search

tinyhttp / cors

🌐 CORS middleware for modern Node.js
MIT License
8 stars 2 forks source link

Refactor origin handler #5

Closed Lordfirespeed closed 2 months ago

Lordfirespeed commented 3 months ago

Resolves #4

Lordfirespeed commented 3 months ago

I've missed a case here - when origin is false an error will be thrown. I'll fix before you guys merge

coveralls commented 2 months ago

Pull Request Test Coverage Report for Build 9908624002

Details

Totals Coverage Status
Change from base Build 7083495285: 0.0%
Covered Lines: 0
Relevant Lines: 0
💛 - Coveralls
talentlessguy commented 2 months ago

Dismiss the report, coverage report was broken

The bug fix is published in 2.0.1, thanks a lot for fixing

https://github.com/tinyhttp/cors/releases/tag/v2.0.1

Coverage is 93%