Skip to content

ENG-1456: add jwt subject support#48

Merged
avesk merged 3 commits intomasterfrom
eng-1456-unit_create_customer_token_via_jwt
Jul 30, 2025
Merged

ENG-1456: add jwt subject support#48
avesk merged 3 commits intomasterfrom
eng-1456-unit_create_customer_token_via_jwt

Conversation

@samuelvasco
Copy link
Collaborator

authorized_users.append(
AuthorizedUser(
data.get("fullName"), data.get("email"), data.get("phone")
data.get("fullName"), data.get("email"), data.get("phone"), data.get("jwtSubject")
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Comment on lines +462 to +465
authorized_user = {"fullName": obj.full_name, "email": obj.email, "phone": obj.phone}
if obj.jwt_subject is not None:
authorized_user["jwtSubject"] = obj.jwt_subject
return authorized_user
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎈

class CreateCustomerToken(UnitRequest):
def __init__(self, customer_id: str, scope: str, verification_token: Optional[str] = None,
verification_code: Optional[str] = None, expires_in: Optional[int] = None):
verification_code: Optional[str] = None, expires_in: Optional[int] = None, jwt_token: Optional[str] = None):
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💪

Copy link
Collaborator

@avesk avesk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@avesk avesk merged commit a0cdf13 into master Jul 30, 2025
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants