Skip to content
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

Jump to bottom

http: improves expect header handling #4501

Closed
wants to merge 1 commit into from
Closed

http: improves expect header handling #4501

wants to merge 1 commit into from

Commits on Jan 8, 2016

  1. http: improves expect header handling 

    Now returns a 417 error status or allows for an event listener on
the `checkExpectation` event. Before we were ignoring requests that
had misspelled `100-continue` values for expect headers.

This is a quick port of the work done here:
nodejs/node-v0.x-archive#7132 by alFReD-NSH
with surrounding discussion here:
nodejs/node-v0.x-archive#4651

Also updates all the instances of the deprecated
EventEmitter.listenerCount to the current self.listenerCount. Most
of these were in the new code ported over but there was another
legacy instance.

Refs: nodejs#2403
    @designfrontier
    designfrontier committed Jan 8, 2016
    Configuration menu
    Copy the full SHA
    810158c View commit details
    Browse the repository at this point in the history