From c0b88514aaaf4ade4a828213805fd2269ee45dc2 Mon Sep 17 00:00:00 2001 From: Qingping Hou Date: Fri, 19 Mar 2021 17:04:42 -0700 Subject: [PATCH] fix: load user info for okta --- flask_appbuilder/security/manager.py | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/flask_appbuilder/security/manager.py b/flask_appbuilder/security/manager.py index ce601c2cd..fe7697007 100644 --- a/flask_appbuilder/security/manager.py +++ b/flask_appbuilder/security/manager.py @@ -618,13 +618,14 @@ def get_oauth_user_info(self, provider, resp): # for Okta if provider == "okta": me = self.appbuilder.sm.oauth_remotes[provider].get("userinfo") - log.debug("User info from Okta: {0}".format(me.data)) + data = me.json() + log.debug("User info from Okta: %s", data) return { - "username": "okta_" + me.data.get("sub", ""), - "first_name": me.data.get("given_name", ""), - "last_name": me.data.get("family_name", ""), - "email": me.data.get("email", ""), - "role_keys": me.data.get("groups", []), + "username": "okta_" + data.get("sub", ""), + "first_name": data.get("given_name", ""), + "last_name": data.get("family_name", ""), + "email": data.get("email", ""), + "role_keys": data.get("groups", []), } else: return {}