◐ Shell
clean mode source ↗

[v6.x] http: fix error check in `Execute()` by mscdex · Pull Request #25939 · nodejs/node

@mscdex added the http

Issues or PRs related to the http subsystem.

label

Feb 5, 2019

@nodejs-github-bot nodejs-github-bot added c++

Issues and PRs that require attention from people who are familiar with C++.

http_parser

Issues and PRs related to the HTTP Parser dependency or the http_parser binding.

v6.x labels

Feb 5, 2019

@addaleax addaleax changed the title http: fix error check in Execute() [v6.x] http: fix error check in Execute()

Feb 9, 2019

@mscdex

BethGriggs

BethGriggs pushed a commit that referenced this pull request

Mar 11, 2019
Refs: #24738
Fixes: #25858

PR-URL: #25939
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Beth Griggs <Bethany.Griggs@uk.ibm.com>

@mscdex mscdex deleted the v6-http-fix-parser-execute-error-check branch

March 11, 2019 16:38

BethGriggs added a commit that referenced this pull request

Mar 15, 2019
Notable changes:

* **http**:
  - fix error check in `Execute()` (Brian White)
    [#25939](#25939)

BethGriggs added a commit that referenced this pull request

Mar 19, 2019
Notable changes:

* **http**:
  - fix error check in `Execute()` (Brian White)
    [#25939](#25939)

BethGriggs added a commit that referenced this pull request

Apr 2, 2019
Notable changes:

- http:
  - fix error check in `Execute()` (Brian White)
    [#25939](#25939)

BethGriggs added a commit that referenced this pull request

Apr 3, 2019
Notable changes:

- http:
  - fix error check in `Execute()` (Brian White)
    [#25939](#25939)

PR-URL: #26684

BethGriggs added a commit that referenced this pull request

Apr 3, 2019
Notable changes:

- http:
  - fix error check in `Execute()` (Brian White)
    [#25939](#25939)

PR-URL: #26684

BethGriggs added a commit that referenced this pull request

Apr 3, 2019
Notable changes:

- http:
  - fix error check in `Execute()` (Brian White)
    [#25939](#25939)

PR-URL: #26684

BethGriggs added a commit that referenced this pull request

Apr 4, 2019
Notable changes:

- http:
  - fix error check in `Execute()` (Brian White)
    [#25939](#25939)

PR-URL: #26684