Re: [whatwg/fetch] Advise `Vary` even for non-CORS-request responses (#564)

annevk commented on this pull request.



>  [[!HTML]]
 [[!HTTP]] [[!HTTP-SEMANTICS]] [[!HTTP-COND]] [[!HTTP-CACHING]] [[!HTTP-AUTH]]
 
-<pre id=example-vary-origin class=example>Vary: Origin</pre>
+<pre id=example-vary-origin class=example>Vary: Origin, Access-Control-Request-Headers, Access-Control-Request-Method</pre>

I don't think it makes sense to include those additional headers, as they're only relevant for the preflight. They're not included in the "actual" request.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/fetch/pull/564#pullrequestreview-50011969

Received on Friday, 14 July 2017 10:17:40 UTC