search

koajs / koa

Expressive middleware for node.js using ES2017 async functions
https://koajs.com
MIT License
35.07k stars 3.22k forks source link

fix: handle upper case protocol like HTTP or HTTPS #1805

Closed FDrag0n closed 5 months ago

FDrag0n commented 5 months ago

https://github.com/koajs/koa/issues/1800 's fix https://github.com/koajs/koa/pull/1804 forgot to consider the case where url is uppercase HTTP and HTTPS

codecov[bot] commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 98.66%. Comparing base (435534a) to head (7f35d51).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1805 +/- ## ======================================= Coverage 98.66% 98.66% ======================================= Files 5 5 Lines 525 525 Branches 147 147 ======================================= Hits 518 518 Misses 7 7 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

FDrag0n commented 5 months ago

@fengmk2 I realized we forgot to deal with Case conversion of http in #1804 fix !