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

refactor(RefreshTokenHandler): Send accessToken in Bearer Header for Refresh Token API and Fix Type Errors #67

Merged
merged 5 commits into from
Feb 14, 2025

Conversation

kimitrii
Copy link
Owner

Changes Made

This PR updates the method of sending the accessToken to the refresh token API by including it in the Authorization bearer header. This change ensures better aligns with standard authentication practices.

Additionally, default empty values were added to the id request parameters to prevent potential type errors. The status type in the errorsHandler was also updated to ContentfulStatusCode to improve type safety and ensure consistency with error handling.

Changes Type

  • Breaking change (Access Token now is send in the Authorization bearer token in the refresh token api)
  • Documentation update -->

Checklist:

  • The changes do not generate new error logs or warnings.
  • I have added tests that prove the fix or new feature works as expected.
  • Both new and existing tests pass locally.

@kimitrii kimitrii merged commit 26e8624 into main Feb 14, 2025
3 checks passed
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.

1 participant