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

Update header for retry invalid session #561

Conversation

zzooeeyy
Copy link
Contributor

@zzooeeyy zzooeeyy commented Jan 8, 2024

Update retry invalid session header so that it won't be stripped

@zzooeeyy zzooeeyy requested a review from a team as a code owner January 8, 2024 21:27
@paulomarg
Copy link
Contributor

Will this change break pre-existing apps that upgrade to this version?

@rezaansyed
Copy link
Contributor

Will this change break pre-existing apps that upgrade to this version?

Nope, this is only part of token exchange so no one is using this header yet.

@paulomarg
Copy link
Contributor

True, I missed that :)

Thought it was the other X- header.

@zzooeeyy
Copy link
Contributor Author

zzooeeyy commented Jan 9, 2024

Closing this because we won't be renaming this header

@zzooeeyy zzooeeyy closed this Jan 9, 2024
@VishnuKP11
Copy link

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.

5 participants