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: simplify checkInvalidHeaderChar #18381

Closed
wants to merge 1 commit into from
Closed

http: simplify checkInvalidHeaderChar #18381

wants to merge 1 commit into from

Commits on Feb 8, 2018

  1. http: simplify checkInvalidHeaderChar 

    In the spirit of [17399](nodejs#17399), we
can also simplify checkInvalidHeaderChar to use regex matching instead
of a loop. This makes it faster on long matches and slower on short
matches or non-matches. This change also includes some sample data from
an AcmeAir benchmark run, as a rough proxy for real-world data.
    @sethbrenith
    sethbrenith committed Feb 8, 2018
    Configuration menu
    Copy the full SHA
    1179bab View commit details
    Browse the repository at this point in the history