Skip to content
This repository has been archived by the owner on Oct 3, 2020. It is now read-only.

Possible ASCII 10 as an only character for line break #15

Closed
jarrodek opened this issue Nov 10, 2016 · 0 comments
Closed

Possible ASCII 10 as an only character for line break #15

jarrodek opened this issue Nov 10, 2016 · 0 comments
Labels

Comments

@jarrodek
Copy link
Collaborator

Here's an example:

[72, 84, 84, 80, 47, 49, 46, 48, 32, 52, 48, 51, 32, 70, 111, 114, 98, 105, 100, 100, 101, 110, 10, 67, 97, 99, 104, 101, 45, 67, 111, 110, 116, 114, 111, 108, 58]

It a response from GitHub API and it means:

HTTP/1.0 403 Forbidden
Cache-Control:

Currently the library is looking for [13, 10, 13, 10] as a line break, then [13, 10] then it takes whole buffer as a headers data (which is correct because it also mean that Chrome socket buffer ended. Then it need to wait for next portion of data,

@jarrodek jarrodek added the bug label Nov 10, 2016
jarrodek added a commit that referenced this issue Nov 10, 2016
…characters to the real value of new lines
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

1 participant