Skip to content

Commit

Permalink
Avoid hardcoding key in auth_state for the user info
Browse files Browse the repository at this point in the history
Co-authored-by: Georgiana <[email protected]>
  • Loading branch information
consideRatio and GeorgianaElena authored Oct 9, 2023
1 parent f66291e commit f084e34
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion config/clusters/callysto/common.values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,7 @@ jupyterhub:
idp_allowed_domains = self.allowed_idps[user_idp].get("allowed_domains")
if idp_allowed_domains:
# this was the part we wanted to replace
email = auth_model["auth_state"]["cilogon_user"]["email"]
email = auth_model["auth_state"][self.user_auth_state_key]["email"]
email_domain = email.split("@", 1)[1].lower()
for ad in idp_allowed_domains:
# fnmatch allow us to use wildcards like * and ?, but
Expand Down

0 comments on commit f084e34

Please sign in to comment.