feat(synapse): use userid as username

uc-cdis · Nov 26, 2019 · 4973f72 · 4973f72
1 parent 9d96c91
commit 4973f72
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/fence/resources/openid/synapse_oauth2.py b/fence/resources/openid/synapse_oauth2.py
@@ -15,10 +15,9 @@ class SynapseOauth2Client(Oauth2ClientBase):
 
     """
 
-    REQUIRED_CLAIMS = {"given_name", "family_name", "email", "email_verified"}
+    REQUIRED_CLAIMS = {"given_name", "family_name", "email", "email_verified", "userid"}
     OPTIONAL_CLAIMS = {
         # "company",
-        # "userid",
         # "orcid",
         # "is_certified",
         # "is_validated",
@@ -125,7 +124,7 @@ def get_user_id(self, code):
                         return dict(error="Required claim {} not found".format(claim))
                 else:
                     rv[claim] = value
-            rv["fence_username"] = rv["email"] + " (via Synapse)"
+            rv["fence_username"] = rv["userid"] + " (via Synapse)"
             return rv
         except Exception as e:
             self.logger.exception("Can't get user info")