Skip to content

http: setEncoding error for incoming socket connections #19344

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
iSkore wants to merge 9 commits into from
Closed

http: setEncoding error for incoming socket connections #19344

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
a7e2aab
http: setEncoding override for incoming packets
iSkore Feb 4, 2018
345ff3c
test: added tests for setEncoding error check
iSkore Feb 4, 2018
4643c63
docs, errors, http: disallow setEncoding on incoming packets
iSkore Mar 6, 2018
f3c429c
docs, errors: note RFC compliance in error message and documentation
iSkore Mar 14, 2018
3613e8d
tests, http: fixed call to ERR_HTTP_INCOMING_SOCKET_ENCODING
iSkore Mar 14, 2018
190ef7e
docs, http: minor format changes per review request
iSkore Mar 29, 2018
06a34eb
docs: update ERR_HTTP_INCOMING_SOCKET_ENCODING per request
iSkore Apr 3, 2018
cd2c3ce
http: minor change per review request
iSkore Apr 19, 2018
a8c605e
Merge branch 'master' into http/issue_18118
iSkore Aug 27, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
http: minor change per review request 
PR-URL: #19344
  • Loading branch information
@iSkore
iSkore committed Apr 19, 2018
commit cd2c3ce6b35206c74b709840191bcff414076ee1
1 change: 0 additions & 1 deletion lib/_http_server.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -384,7 +384,6 @@ function connectionListenerInternal(server, socket) {

// Override on to unconsume on `data`, `readable` listeners
socket.on = socketOnWrap;

socket.setEncoding = socketSetEncoding;
Copy link
Copy Markdown
Member

@TimothyGu TimothyGu Mar 29, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: either a comment or a new line here would be nice, since this line isn't related to

  // Override on to unconsume on `data`, `readable` listeners

Copy link
Copy Markdown
Contributor Author

@iSkore iSkore Mar 29, 2018 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree, because I didn't see that done in errors.md(aside from internal page links) I tried to follow suit. I'll update that.


// We only consume the socket if it has never been consumed before.
Expand Down