search

jonnyreeves / fetch-readablestream

Compatibility layer for efficient streaming of binary data using WHATWG Streams
MIT License
49 stars 10 forks source link

[FIX] fix crlf to split response headers correctly #13

Open leesnhyun opened 4 years ago

leesnhyun commented 4 years ago

OS : Windows 10, IE 11 (v 11.450.190410.0)

Sometimes CRLF of response header string is only '\n' at old browser (in my case, IE11).

To prevent unhandled exception, add some defense code by changing from "\r\n" to /\r|\n|\r\n/ .

jonnyreeves commented 4 years ago

Thanks for raising a fix for this @leesnhyun, I haven't touched this project for a long time but it does appear to have some unit tests, can you please confirm they are still passing and consider adding a new one to cover this regex? Thanks!