Remove extraneous logging arg in authform login plugin
authorRob Crittenden <rcritten@redhat.com>
Fri, 8 May 2015 13:46:24 +0000 (09:46 -0400)
committerRob Crittenden <rcritten@redhat.com>
Fri, 8 May 2015 14:02:36 +0000 (10:02 -0400)
If you didn't provide credentials at all at the form by
pressing ENTER then a 500 error would be thrown rather
than an authentication error.

Signed-off-by: Rob Crittenden <rcritten@redhat.com>
Reviewed-by: Patrick Uiterwijk <puiterwijk@redhat.com>
ipsilon/login/authform.py

index 0e20a60..55dc6b3 100644 (file)
@@ -23,7 +23,6 @@ from ipsilon.util import config as pconfig
 from string import Template
 import cherrypy
 import subprocess
-import logging
 
 
 class Form(LoginFormBase):
@@ -39,7 +38,7 @@ class Form(LoginFormBase):
                 error = cherrypy.request.headers['EXTERNAL_AUTH_ERROR']
             except KeyError:
                 error = "Unknown error using external authentication"
-                cherrypy.log.error("Error: %s" % error, logging.ERROR)
+                cherrypy.log.error("Error: %s" % error)
             return self.lm.auth_failed(self.trans)