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

add vary origin header for specifc origins #30

Merged
merged 1 commit into from
Oct 23, 2014

Conversation

soarez
Copy link
Contributor

@soarez soarez commented Oct 22, 2014

If the server specifies an origin host rather than "*", then it must
also include Origin in the Vary response header to indicate to clients
that server responses will differ based on the value of the Origin
request header.

https://developer.mozilla.org/en-US/docs/Web/HTTP/Access_control_CORS#Access-Control-Allow-Origin

If the server specifies an origin host rather than "*", then it must
also include Origin in the Vary response header to indicate to clients
that server responses will differ based on the value of the Origin
request header.

https://developer.mozilla.org/en-US/docs/Web/HTTP/Access_control_CORS#Access-Control-Allow-Origin
troygoode added a commit that referenced this pull request Oct 23, 2014
add vary origin header for specifc origins
@troygoode troygoode merged commit b0e3ae3 into expressjs:master Oct 23, 2014
@troygoode
Copy link
Member

nice catch

@troygoode
Copy link
Member

thanks @soarez. this is now on npm as 2.5.0

@dougwilson
Copy link
Contributor

This will remove any previously-set Vary header value now, no? Why not use the vary module :)?

troygoode added a commit that referenced this pull request Nov 7, 2014
@troygoode
Copy link
Member

great suggestion @dougwilson (and thanks for writing the module :-)) - I've published this as 2.5.1

@dougwilson
Copy link
Contributor

yay! I'm glad you added the vary header--I was actually manually adding it myself, but now I can remove that by upgrading the module :D

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.

3 participants