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

Legg til Vary: X-Json-Serializer-Option header på alle responser. #242

Merged
merged 1 commit into from
Mar 20, 2025

Conversation

josstn
Copy link
Contributor

@josstn josstn commented Mar 17, 2025

Behov / Bakgrunn

Uten slik Vary header kunne respons bli cacha av klient og brukt opp igjen ved ny request til samme url, sjølv om X-Json-Serializer-Option vart endra mellom requests.

Dette ville føre til feil resultat for klient.

Løsning

Legger til Vary header på alle responser frå serveren.

Slik at det ikkje blir feil viss ein klient cacher respons og endrer header mellom requests.
@josstn josstn requested a review from a team as a code owner March 17, 2025 21:30
@josstn josstn merged commit aca39f5 into master Mar 20, 2025
8 checks passed
@josstn josstn deleted the vary-header-alle branch March 20, 2025 09:28
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.

2 participants