Skip to content

Commit a00523d

Browse files
committed
conform str to user id
1 parent 43933e5 commit a00523d

File tree

3 files changed

+7
-7
lines changed

3 files changed

+7
-7
lines changed

fence/jwt/token.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -179,7 +179,7 @@ def generate_signed_session_token(kid, private_key, expires_in, context=None):
179179
claims = {
180180
"pur": "session",
181181
"aud": ["fence", issuer],
182-
"sub": context.get("user_id", ""),
182+
"sub": str(context.get("user_id", "")),
183183
"iss": issuer,
184184
"iat": iat,
185185
"exp": exp,

tests/link/test_link.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,7 @@ def test_google_link_session(app, client, encoded_creds_jwt):
139139
)
140140

141141
assert flask.session.get("google_link") is True
142-
assert flask.session.get("user_id") == user_id
142+
assert flask.session.get("user_id") == str(user_id)
143143
assert flask.session.get("google_proxy_group_id") == proxy_group_id
144144
assert flask.session.get("redirect") == redirect
145145

tests/utils/__init__.py

+5-5
Original file line numberDiff line numberDiff line change
@@ -287,7 +287,7 @@ def unauthorized_context_claims(user_name, user_id):
287287
iat, exp = iat_and_exp()
288288
return {
289289
"aud": [iss],
290-
"sub": user_id,
290+
"sub": str(user_id),
291291
"pur": "access",
292292
"iss": iss,
293293
"iat": iat,
@@ -320,7 +320,7 @@ def authorized_download_context_claims(user_name, user_id):
320320
iat, exp = iat_and_exp()
321321
return {
322322
"aud": [iss],
323-
"sub": user_id,
323+
"sub": str(user_id),
324324
"iss": iss,
325325
"iat": iat,
326326
"exp": exp,
@@ -353,7 +353,7 @@ def authorized_service_account_management_claims(user_name, user_id, client_id):
353353
iat, exp = iat_and_exp()
354354
return {
355355
"aud": [iss],
356-
"sub": user_id,
356+
"sub": str(user_id),
357357
"iss": iss,
358358
"iat": iat,
359359
"exp": exp,
@@ -403,7 +403,7 @@ def authorized_download_credentials_context_claims(
403403
iat, exp = iat_and_exp()
404404
return {
405405
"aud": [iss],
406-
"sub": user_id,
406+
"sub": str(user_id),
407407
"iss": iss,
408408
"iat": iat,
409409
"exp": exp,
@@ -436,7 +436,7 @@ def authorized_upload_context_claims(user_name, user_id):
436436
iat, exp = iat_and_exp()
437437
return {
438438
"aud": [iss],
439-
"sub": user_id,
439+
"sub": str(user_id),
440440
"iss": iss,
441441
"pur": "access",
442442
"iat": iat,

0 commit comments

Comments
 (0)