cors-middleware icon indicating copy to clipboard operation
cors-middleware copied to clipboard

Update neomerx/cors-psr7 to v2

Open internalsystemerror opened this issue 5 years ago • 1 comments

Version 2 is out https://github.com/neomerx/cors-psr7/tree/v2.0.2. I can see you're already ahead on this: https://github.com/neomerx/cors-psr7/issues/32. Just wanted to give you the heads up.

internalsystemerror avatar Mar 27 '19 12:03 internalsystemerror

As a side note on the neomerx/cors-psr7 update, I've detected a bug in the method Analizer->getRequestedHeadersInLowerCase($request) it only gets the first of the methods of the request, so at the implementation of the method isRequestAllHeadersSupported of the AnalysisStrategyInterface interface in version v1.0.13 of neomerx/cors-psr7, only the first header of the request is checked against the headers.allow array of the CorsMiddleware configuration options. The version 2.x effectively corrects the issue with a forEach loop of all the request headers.

Rodrirokr avatar Sep 02 '19 14:09 Rodrirokr