Skip to content

[v14.x] Revert "http: lazy create IncomingMessage.headers" - #36553

Closed
BethGriggs wants to merge 1 commit into
nodejs:v14.x-stagingfrom
BethGriggs:v14.x-revert-35281
Closed

[v14.x] Revert "http: lazy create IncomingMessage.headers"#36553
BethGriggs wants to merge 1 commit into
nodejs:v14.x-stagingfrom
BethGriggs:v14.x-revert-35281

Conversation

@BethGriggs

@BethGriggsBethGriggs commented Dec 17, 2020

Copy link
Copy Markdown
Member

This reverts commit b58725c.

Fixes: #36550
Refs: #35281

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

@nodejs-github-botnodejs-github-bot added http Issues or PRs related to the http subsystem. v14.x labels Dec 17, 2020
@BethGriggsBethGriggs changed the title Revert "http: lazy create IncomingMessage.headers"[v14.x] Revert "http: lazy create IncomingMessage.headers"Dec 17, 2020
@BethGriggsBethGriggs added the request-ci Add this label to start a Jenkins CI on a PR. label Dec 17, 2020
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Dec 17, 2020
@nodejs-github-bot

nodejs-github-bot commented Dec 17, 2020

Copy link
Copy Markdown
Collaborator

@MylesBorinsMylesBorins left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

RSLGTM

@mhdawsonmhdawson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@mcollinamcollina left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@BethGriggs

Copy link
Copy Markdown
MemberAuthor

Landed in 29fac46

BethGriggs added a commit that referenced this pull request Dec 17, 2020
This reverts commit b58725c.
Fixes: #36550
PR-URL: #36553
Refs: #35281
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
@BethGriggsBethGriggs mentioned this pull request Dec 17, 2020
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

httpIssues or PRs related to the http subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@BethGriggs@nodejs-github-bot@mcollina@MylesBorins@richardlau@mhdawson@aduh95