cors
cors copied to clipboard
feat: Add callback for configuring Vary header
Fixes #317
This PR adds a callback (shouldSetVaryHeader
) to the options which is used to determine when and how to configure the Vary header.
To preserve backward compatibility with all existing cors
usages, this callback returns true
by default.
If this PR is accepted, I will add documentation to the README on how to use this PR's functionality.
I enabled the CI to run in this PR :+1:
Hey @gl-jkeys 👋, first of all, thank you for being an awesome contributor.
If this PR is accepted, I will add documentation to the README on how to use this PR's functionality.
It makes more sense if that README change is also included in this PR. Could you please take care of that?
(Note to self: I was unable to run the tests due to my own PC issue, I will check the tests again sometime later soon).
Hey @IamLizu, I made the changes you requested; I also modified the PR to unconditionally set the Vary: Origin header for OPTIONS requests. (It seems unlikely, in hindsight since this PR was opened, that OPTIONS requests would be cached without respect to Origin.) I also simplified the tests to match the updated implementation.