-
-
Notifications
You must be signed in to change notification settings - Fork 682
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for multiple issuers #627
Conversation
for more information, see https://pre-commit.ci
@@ -212,7 +212,10 @@ def _validate_iss(self, payload, issuer): | |||
if "iss" not in payload: | |||
raise MissingRequiredClaimError("iss") | |||
|
|||
if payload["iss"] != issuer: | |||
if isinstance(issuer, str): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we also need to guard against other invalid values for issuer
?
see https://github.com/jpadilla/pyjwt/blob/master/jwt/api_jwt.py#L128-L129
I'm interested in this feature. |
Co-authored-by: Dima Tisnek <[email protected]>
for more information, see https://pre-commit.ci
That makes two of us. |
@jpadilla is support for multiple issuers coming soon? |
there were some unresolved comments on this pull request, I'd gladly review again another. |
Closes #433
This PR adds support for specifing multiple issuers, similar to what we do for audience.