Add allowMethods options to @middy/http-cors

[camiloaz]

What does this implement/fix? Explain your changes.

This PR adds the option to set an allowMethods option that allows setting the Access-Control-Allow-Methods header in the preflight CORS requests.

Does this close any currently open issues?

There is an already closed issue #43 that asks for this. But it was never really solved.

Where has this been tested?

Node.js Versions: 12.18.3

Middy Versions: 1.5.2

AWS SDK Versions:

Todo list

[x] Feature/Fix fully implemented
[x] Added tests
[x] Updated relevant documentation
[x] Updated relevant examples

[camiloaz]

BTW, I noticed that on npm install all the package-lock.json files changed. The root package-lock.json changes only on this line:

And then all the packages include changes like this:

I wonder whether i should commit that or what's wrong there?

[willfarrell]

@camiloaz thanks for the PR. We're no longer accepting changes to v1.

Access-Control-Allow-Methods has been added to v2 which is set to have a stable release soon.

https://github.com/middyjs/middy/blob/release/2.x/packages/http-cors/index.js#L54

Please let us know if this implementation meets your needs.