Remove redundant check for user_groups

asfadmin · Jan 27, 2022 · 4e2f7fc · 4e2f7fc
1 parent f9822ca
commit 4e2f7fc
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/rain_api_core/urs_util.py b/rain_api_core/urs_util.py
@@ -319,8 +319,7 @@ def do_login(args, context, cookie_domain='', aux_headers=None):
         else:
             redirect_to = get_base_url(context)
 
-        # TODO(reweeden): Why is the last check there if we're setting to the empty list anyways?
-        if 'user_groups' not in user_profile or not user_profile['user_groups']:
+        if 'user_groups' not in user_profile:
             user_profile['user_groups'] = []
 
         jwt_cookie_payload = user_profile_2_jwt_payload(user_id, auth['access_token'], user_profile)