[GitHub] Enalmada commented on issue #1760: ModPagespeedAddResourceHeader Access-Control-Allow-Origin duplicate headers

[GitBox <gi...@apache.org>]

Enalmada commented on issue #1760: ModPagespeedAddResourceHeader Access-Control-Allow-Origin duplicate headers
URL: https://github.com/apache/incubator-pagespeed-mod/issues/1760#issuecomment-377579188
 
 
   Here is a test server that shows the dupe headers bug: https://test.gelltest.com/home
   Purge is public for the test: curl -k --request PURGE 'https://test.gelltest.com/*'
   
   I tried using mod_headers merge but it doesn't seem to touch the duplicate.  I may have stumbled on a workaround though...disabling ModPagespeedAddResourceHeaders and using Header setifempty so far seems to be working.  So for anyone else using apache with origin returning cors and also needing it on optimized assets, this could be a workaround: 
   `# Do not enable ModPagespeedAddResourceHeaders Access-Control-Allow-Origin`
   `Header setifempty Access-Control-Allow-Origin <MY_ORIGIN>`
   
   This ticket is a request to make ModPagespeedAddResourceHeader not add a header if it already exists on the response.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services