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

Auth now specifies api as audience. #2292

Merged
merged 1 commit into from
Jan 8, 2025
Merged

Conversation

mikesmit
Copy link
Collaborator

@mikesmit mikesmit commented Jan 6, 2025

partially fixes #2263

In the baseline implementation of the app the auth token request did not specify an audience (the target API for the JWT). After defining the policyengine API in auth0, I have now updated both the provider and the fetch with auth to specify issuer.

Immediately this will actually stop existing, logged in users from submitting a bearer token until they have to re-log in.

Since the bearer token is yet used this change will not impact the user experience.

partially fixes #2263

In the baseline implementation of the app the auth token request did not
specify an audience (the target API for the JWT). After defining the
policyengine API in auth0, I have now updated both the provider and the
fetch with auth to specify issuer.

Immediately this will actually stop existing, logged in users from
submitting a bearer token until they have to re-log in.

Since the bearer token is yet used this change will not impact the user
experience.
@MaxGhenis MaxGhenis merged commit 4bfece9 into master Jan 8, 2025
2 checks passed
@MaxGhenis MaxGhenis deleted the 2263_add_aud_to_bearer_token branch January 8, 2025 21:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

App should forward authentication token to the API on all requests
2 participants