search

monsur / enable-cors.org

Advocacy for CORS
http://enable-cors.org/
212 stars 99 forks source link

Change Allow and Expose headers in nginx config #133

Closed ravishivt closed 6 years ago

ravishivt commented 6 years ago

I assume a lot of people blindly copy your configs and paste them in without review. For the nginx config provided, I think there are some optimizations that can be made in regards to the Access-Control-Allow-Headers and Access-Control-Expose-Headers settings. See my list of suggested changes and the reasoning behind them at this stack overflow post.

monsur commented 6 years ago

Thanks Ravi. Would you be willing to create a pull request for this? Thanks!

ravishivt commented 6 years ago

Sure, PR #134 created.