Howard86 / next-api-handler

lightweight nextjs api handler wrapper, portable & configurable for serverless environment
https://next-api-handler.vercel.app
Other
49 stars 1 forks source link

Bugfix/add cors integration #93

Closed Howard86 closed 1 year ago

Howard86 commented 1 year ago

Bug fix

Fix when adding cors middleware, OPTIONS returns 405 which is unnecessary

Only run router handler if its a supported method

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
next-api-handler ✅ Ready (Inspect) Visit Preview Apr 18, 2023 6:12am
next-api-handler-example ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 18, 2023 6:12am
codeclimate[bot] commented 1 year ago

Code Climate has analyzed commit e40d27e4 and detected 0 issues on this pull request.

View more on Code Climate.

cypress[bot] commented 1 year ago

Passing run #166 ↗︎

0 10 0 0 Flakiness 0

Details:

null
Project: next-api-handler Commit: e40d27e47f
Status: Passed Duration: 00:09 💡
Started: Apr 18, 2023 6:14 AM Ended: Apr 18, 2023 6:14 AM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

codecov[bot] commented 1 year ago

Codecov Report

Merging #93 (e40d27e) into main (62058c9) will increase coverage by 0.00%. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #93   +/-   ##
=======================================
  Coverage   99.84%   99.84%           
=======================================
  Files           8        8           
  Lines         636      652   +16     
  Branches       85       88    +3     
=======================================
+ Hits          635      651   +16     
  Misses          1        1           

see 4 files with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more