saml2: downgrade some errors to warning or info level (#18369)
This commit is contained in:
parent
25e6b3e7e5
commit
e081de62f3
|
@ -154,7 +154,7 @@ class Saml2Directory(Directory):
|
|||
return Directory._q_traverse(self, path)
|
||||
|
||||
def log_profile_error(self, profile, error, what):
|
||||
get_logger().error('%(what)s from %(provider)s failed: %(message)s' % {
|
||||
get_logger().info('%(what)s from %(provider)s failed: %(message)s' % {
|
||||
'provider': profile.remoteProviderId,
|
||||
'message': error[1], 'what': what })
|
||||
|
||||
|
@ -243,7 +243,7 @@ class Saml2Directory(Directory):
|
|||
elif request.get_method() == 'POST':
|
||||
message, method = request.form.get('SAMLart',None), lasso.HTTP_METHOD_ARTIFACT_POST
|
||||
else:
|
||||
get_logger().error('Bad HTTP method on assertionConsumerArtifact endpoint')
|
||||
get_logger().info('Bad HTTP method on assertionConsumerArtifact endpoint')
|
||||
return error_page(_('Invalid authentication response'))
|
||||
login.initRequest(message, method)
|
||||
except lasso.Error, error:
|
||||
|
@ -743,7 +743,7 @@ class Saml2Directory(Directory):
|
|||
if sessions:
|
||||
logout.setSessionFromDump(sessions[0].lasso_session_dump)
|
||||
else:
|
||||
get_logger().error('No session open for nameid %s', logout.nameIdentifier.content)
|
||||
get_logger().info('No session open for nameid %s', logout.nameIdentifier.content)
|
||||
return self.slo_idp_finish(logout, soap)
|
||||
user = sessions[0] and sessions[0].get_user()
|
||||
if user and user.lasso_dump:
|
||||
|
@ -757,7 +757,7 @@ class Saml2Directory(Directory):
|
|||
self.kill_sessions(logout.nameIdentifier, logout.request.sessionIndexes)
|
||||
except Exception, error:
|
||||
get_session_manager().expire_session()
|
||||
get_logger().error('kill_session: %s' % error)
|
||||
get_logger().info('kill_session: %s' % error)
|
||||
return self.slo_idp_finish(logout, soap)
|
||||
|
||||
# if session is still not good, clean it
|
||||
|
|
Loading…
Reference in New Issue