getAllResponseHeaders does not include trailing CRLF

[jsedlacek]

According to spec, the getAllResponseHeaders should include trailing CRLF:
https://xhr.spec.whatwg.org/#the-getallresponseheaders()-method

Xdomains implementation does not contain trailing CRLF.
https://github.com/jpillora/xdomain/blob/gh-pages/vendor/xhook.js#L226

This breaks behavior of superagent, which pops the trailing CRLF:
https://github.com/visionmedia/superagent/blob/master/lib/client.js#L203

With xdomain trailing CRLF is not present, so the last header gets stripped.

[jpillora]

Thanks, this should be fixed in jpillora/xhook

…

On Fri, 8 Sep 2017 at 5:44 am Jakub Sedlacek ***@***.***> wrote: According to spec, the getAllResponseHeaders should include trailing CRLF: https://xhr.spec.whatwg.org/#the-getallresponseheaders()-method Xdomains implementation does not contain trailing CRLF. https://github.com/jpillora/xdomain/blob/gh-pages/vendor/xhook.js#L226 This breaks behavior of superagent, which pops the trailing CRLF: https://github.com/visionmedia/superagent/blob/master/lib/client.js#L203 With xdomain trailing CRLF is not present, so the last header gets stripped. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#202>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAmr81wkQTF83qt2hTyjgcyxq6mUQh26ks5sgEeqgaJpZM4PQTI3> .

[jsedlacek]

Thanks, I have created the upstream issue as well.
jpillora/xhook#72