Skip to content

Commit

Permalink
fix name convention ( was still wrong )
Browse files Browse the repository at this point in the history
  • Loading branch information
spigad authored Jun 1, 2022
1 parent a78d6ae commit 1970081
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions docker/single-node-jupyterhub/jupyterhub/jupyterhub_config.py
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ def pre_spawn_start(self, user, spawner):
self.log.info(auth_state["oauth_user"]["groups"])
self.log.info(allowed_groups)

matched_groups = set(allowed_groups_full).intersection(set(auth_state["oauth_user"]["groups"]))
matched_groups = set(allowed_groups).intersection(set(auth_state["oauth_user"]["groups"]))
if matched_groups: amIAllowed = True

if not amIAllowed:
Expand Down Expand Up @@ -137,7 +137,7 @@ async def authenticate(self, handler, data=None):
# allowed_admin_groups = [ele for ele in allowed_admin_groups_full if not re.search('catchall', ele)]
allowed_admin_groups = os.environ["ADMIN_OAUTH_GROUPS"].split(" ")

matched_admin_groups = set(allowed_admin_groups_full).intersection(set(auth_state["oauth_user"]["groups"]))
matched_admin_groups = set(allowed_admin_groups).intersection(set(auth_state["oauth_user"]["groups"]))

if os.environ.get("OAUTH_SUB") == auth_state["oauth_user"]["sub"] or matched_admin_groups:
self.log.info(
Expand Down

0 comments on commit 1970081

Please sign in to comment.