Skip to content

Commit

Permalink
callysto: 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 committed Oct 9, 2023
1 parent 8cddc64 commit 14d4b5e
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 14d4b5e

Please sign in to comment.