Skip to content
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

Aud(ience) is not verified #40

Open
spawn-guy opened this issue May 11, 2021 · 4 comments
Open

Aud(ience) is not verified #40

spawn-guy opened this issue May 11, 2021 · 4 comments
Labels
enhancement New feature or request released Changes was released for this

Comments

@spawn-guy
Copy link

i can't seem to find an audience (and the rest params) verifier.
it seems that only signature is verified

i see a decode method that should do all that and get the needed information back here https://github.com/mpdavis/python-jose/blob/master/jose/jwt.py#L57

can this be used?

@tokusumi
Copy link
Owner

@spawn-guy Thank you for your issue and proposing solution.
This is fixed at #45

@tokusumi tokusumi added enhancement New feature or request released Changes was released for this labels Jun 13, 2021
@jleclanche
Copy link
Contributor

@tokusumi Hi! I just upgraded to 0.4.0. I'm using auth0 and I see that customAPI is now a required parameter to Auth0().

I understand this will essentially validate the client ID that is trying to authenticate. Now, what if there are multiple valid client IDs we want to authenticate against?

One scenario I could see: An android app, an iOS app and an electron app all three with different client IDs, trying to auth against the same endpoint.

This is a bit confusing and not made clear in the docs (it just says "audience"). WDYT?

@spawn-guy
Copy link
Author

@jleclanche you need to validate against server keys, not the client keys.

Client gives you a token, and server validates that it can accept the token. The token is valid an the token allows access to this server.

@jleclanche
Copy link
Contributor

@spawn-guy Thanks for the pointers. I think this could use some examples for auth0; I will try to figure out how to do it properly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request released Changes was released for this
Projects
None yet
Development

No branches or pull requests

3 participants