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

Use Authorization header instead of Authentication #2291

Merged
merged 1 commit into from
Jan 3, 2025

Conversation

mikesmit
Copy link
Collaborator

@mikesmit mikesmit commented Jan 3, 2025

partially addresses #2263

updated useAuthenticatedFetch to correctly set the "Authorization" header to the bearer token instead of
"Authentication"

@mikesmit mikesmit requested a review from anth-volk January 3, 2025 01:16
@MaxGhenis
Copy link
Contributor

Think the title should be flipped?

@mikesmit
Copy link
Collaborator Author

mikesmit commented Jan 3, 2025

Think the title should be flipped?

Demonstrating how this happened in the first place.... Fixing title.

partially addresses #2263

updated useAuthenticatedFetch to correctly set the "Authorization" header to the bearer token instead of
"Authentication"
@mikesmit mikesmit changed the title Use Authentication not Authorzation in header Use Authorization header instead of Authentication Jan 3, 2025
@anth-volk
Copy link
Collaborator

Just found where auth0 specifies this in the docs here. Thanks, will approve and merge!

@anth-volk anth-volk merged commit 3595205 into master Jan 3, 2025
2 checks passed
@anth-volk anth-volk deleted the 2263_fix_header branch January 3, 2025 13:34
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.

3 participants