Logger cleanup

This commit is contained in:
Rachid Mrad 2023-12-12 19:31:09 -05:00
parent 2f47272fff
commit 18e215dd08
No known key found for this signature in database
GPG key ID: EF38E4CEC4A8F3CF

View file

@ -87,7 +87,7 @@ class Client(oic.Client):
extra_args=None, extra_args=None,
): ):
"""Step 2: Construct a login URL at OP's domain and send the user to it.""" """Step 2: Construct a login URL at OP's domain and send the user to it."""
logger.debug("create_authn_request() Creating the OpenID Connect authn request...") logger.debug("Creating the OpenID Connect authn request...")
state = rndstr(size=32) state = rndstr(size=32)
try: try:
session["state"] = state session["state"] = state
@ -150,7 +150,7 @@ class Client(oic.Client):
def callback(self, unparsed_response, session): def callback(self, unparsed_response, session):
"""Step 3: Receive OP's response, request an access token, and user info.""" """Step 3: Receive OP's response, request an access token, and user info."""
logger.debug("callback() Processing the OpenID Connect callback response...") logger.debug("Processing the OpenID Connect callback response...")
state = session.get("state", "") state = session.get("state", "")
try: try:
# parse the response from OP # parse the response from OP
@ -174,7 +174,7 @@ class Client(oic.Client):
logger.error("Unable to process response %s for %s" % (error, state)) logger.error("Unable to process response %s for %s" % (error, state))
raise o_e.AuthenticationFailed(locator=state) raise o_e.AuthenticationFailed(locator=state)
logger.debug("callback() authn_response %s" % authn_response) logger.debug("authn_response %s" % authn_response)
if not authn_response.get("state", None): if not authn_response.get("state", None):
logger.error("State value not received from OP for %s" % state) logger.error("State value not received from OP for %s" % state)
@ -213,7 +213,7 @@ class Client(oic.Client):
logger.error("Unable to get user info (%s) for %s" % (info_response.get("error", ""), state)) logger.error("Unable to get user info (%s) for %s" % (info_response.get("error", ""), state))
raise o_e.AuthenticationFailed(locator=state) raise o_e.AuthenticationFailed(locator=state)
logger.debug("_get_user_info() user info: %s" % info_response) logger.debug("user info: %s" % info_response)
return info_response.to_dict() return info_response.to_dict()