Merge pull request #3642 from yuvipanda/auth0-none-check

Handle deployment-service-check user for auth0
2i2c-org · Jan 25, 2024 · 9a1433d · 9a1433d
2 parents 5c8194c + e986799
commit 9a1433d
Showing 1 changed file with 9 additions and 6 deletions.
diff --git a/config/clusters/earthscope/common.values.yaml b/config/clusters/earthscope/common.values.yaml
@@ -87,12 +87,15 @@ basehub:
               return False
 
           def populate_token(spawner, auth_state):
-            token_env = {
-              'AUTH0_ACCESS_TOKEN': auth_state.get("access_token", ""),
-              'AUTH0_ID_TOKEN': auth_state.get("id_token", ""),
-              'AUTH0_REFRESH_TOKEN': auth_state.get('refresh_token', '')
-            }
-            spawner.environment.update(token_env)
+            # For our deployment-service-check health check user, there is no auth_state.
+            # So these env variables need not be set.
+            if auth_state:
+              token_env = {
+                'AUTH0_ACCESS_TOKEN': auth_state.get("access_token", ""),
+                'AUTH0_ID_TOKEN': auth_state.get("id_token", ""),
+                'AUTH0_REFRESH_TOKEN': auth_state.get('refresh_token', '')
+              }
+              spawner.environment.update(token_env)
 
           c.Spawner.auth_state_hook = populate_token