-
Notifications
You must be signed in to change notification settings - Fork 58
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
Vary with dynamic config #291
Comments
This was referenced Jan 27, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Prerequisites
Last working version
8.x.x
Stopped working in version
9.x.x
Node.js version
any
Operating system
Linux
Operating system version (i.e. 20.04, 11.3, 10)
any
💥 Regression Report
this testcase has two different origins (example.com and sample.com):
fastify-cors/test/cors.test.js
Lines 126 to 221 in 3cf8f61
But does not add the Vary: origin header.
Steps to Reproduce
add expectation that the vary header exists back to that test (and the other tests with config delegation changed in #288)
Expected Behavior
No response
The text was updated successfully, but these errors were encountered: