Skip to content

Commit

Permalink
Merge pull request #5244 from minrk/refresh_user_fix
Browse files Browse the repository at this point in the history
jupyter-health: reapply refresh_user fix
  • Loading branch information
minrk authored Dec 6, 2024
2 parents 6887147 + 6bd1706 commit c98bb31
Showing 1 changed file with 12 additions and 0 deletions.
12 changes: 12 additions & 0 deletions config/clusters/jupyter-health/common.values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -105,6 +105,18 @@ jupyterhub:
spawner.environment["JHE_CLIENT_ID"] = "Ima7rx8D6eko0PzlU1jK28WBUT2ZweZj7mqVG2wm"
c.Spawner.auth_state_hook = auth_state_env
skip_refresh_for_test_user.py: |
def refresh_user_hook(authenticator, user, auth_state):
if user.name == "deployment-service-check":
# if this is the user,
# refresh_user doesn't make sense
# consider it always fresh
return True
# for all other users, refresh as usual
return None
c.OAuthenticator.refresh_user_hook = refresh_user_hook
singleuser:
defaultUrl: /lab
extraEnv:
Expand Down

0 comments on commit c98bb31

Please sign in to comment.