search

jdesrosiers / silex-cors-provider

A silex service provider that adds CORS services to silex
MIT License
78 stars 25 forks source link

Fix multiple allowed CORS methods in Internet Explorer Edge #19

Closed cyberwolf closed 9 years ago

cyberwolf commented 9 years ago

Internet Explorer Edge does not like multiple Access-Control-Allow-Methods headers, instead put all values in a comma separated list in one header.

jdesrosiers commented 9 years ago

Thanks for finding this. Can you also add or modify a unit test to enforce this?

cyberwolf commented 9 years ago

@jdesrosiers, sure, I've added a unit test.

jdesrosiers commented 9 years ago

Merged manually into v0 and master