Skip to content

deps: update undici to 5.25.2#49559

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
Merged
nodejs-github-bot merged 3 commits into from
Sep 23, 2023
Merged

deps: update undici to 5.25.2 #49559

nodejs-github-bot merged 3 commits into from
Sep 23, 2023

Conversation

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-botnodejs-github-bot commented Sep 8, 2023•
edited
Loading

This is an automated update of undici to 5.25.2.

@nodejs-github-botnodejs-github-bot added dependencies Pull requests that update a dependency file. lib / src Issues and PRs related to general changes in the lib or src directory. needs-ci PRs that need a full CI run. labels Sep 8, 2023
@mcollinamcollina added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 8, 2023
@github-actionsgithub-actionsbot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 8, 2023
@nodejs-github-bot
Copy link
CollaboratorAuthor

nodejs-github-bot commented Sep 8, 2023

CI: https://ci.nodejs.org/job/node-test-pull-request/53819/

@aduh95
Copy link
Contributor

aduh95 commented Sep 8, 2023

Quite a few seemingly related test failures

@KhafraDev
Copy link
Member

KhafraDev commented Sep 8, 2023

is it only builds without ssl?

This was referenced Sep 9, 2023
Open
Open
Open
@joyeecheungjoyeecheung mentioned this pull request Sep 11, 2023
Open
@github-actionsgithub-actionsbot mentioned this pull request Sep 12, 2023
Open
19 tasks
@atlowChemiatlowChemi mentioned this pull request Sep 13, 2023
Closed
@atlowChemiatlowChemi linked an issue Sep 13, 2023 that may be closed by this pull request
Unexpected error when using signal with reason in fetch #49557
Closed
@KhafraDevKhafraDev mentioned this pull request Sep 19, 2023
Merged
@nodejs-github-botnodejs-github-bot changed the title deps: update undici to 5.24.0deps: update undici to 5.25.0Sep 20, 2023
@mcollinamcollina added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 21, 2023
mcollina
mcollina approved these changes Sep 21, 2023
View reviewed changes
Copy link
Member

@mcollinamcollina left a comment

Choose a reason for hiding this comment

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

lgtm

@github-actionsgithub-actionsbot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 21, 2023
@nodejs-github-bot
Copy link
CollaboratorAuthor

nodejs-github-bot commented Sep 21, 2023

CI: https://ci.nodejs.org/job/node-test-pull-request/54106/

@github-actionsgithub-actionsbot mentioned this pull request Sep 22, 2023
Open
17 tasks
@nodejs-github-botnodejs-github-bot changed the title deps: update undici to 5.25.0deps: update undici to 5.25.2Sep 22, 2023
@mcollina
Copy link
Member

mcollina commented Sep 22, 2023

@KhafraDev let's see if WebSocket runs :)

@KhafraDev
Copy link
Member

KhafraDev commented Sep 22, 2023

I tested it locally, there is something very strange going on if it doesn't work lol

@mcollinamcollina added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 23, 2023
@github-actionsgithub-actionsbot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 23, 2023
@nodejs-github-bot
Copy link
CollaboratorAuthor

nodejs-github-bot commented Sep 23, 2023

CI: https://ci.nodejs.org/job/node-test-pull-request/54170/

@github-actionsgithub-actionsbot mentioned this pull request Sep 23, 2023
Open
16 tasks
@nodejs-github-bot
Copy link
CollaboratorAuthor

nodejs-github-bot commented Sep 23, 2023

CI: https://ci.nodejs.org/job/node-test-pull-request/54175/

@mcollinamcollina added commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. commit-queue Add this label to land a pull request using GitHub Actions. and removed needs-ci PRs that need a full CI run. labels Sep 23, 2023
mcollina
mcollina approved these changes Sep 23, 2023
View reviewed changes
Copy link
Member

@mcollinamcollina left a comment

Choose a reason for hiding this comment

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

lgtm

@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 23, 2023
@nodejs-github-botnodejs-github-bot merged commit ef062b9 into mainSep 23, 2023
@nodejs-github-botnodejs-github-bot deleted the actions/tools-update-undici branch September 23, 2023 11:40
@nodejs-github-bot
Copy link
CollaboratorAuthor

nodejs-github-bot commented Sep 23, 2023

Landed in ef062b9

@github-actionsgithub-actionsbot mentioned this pull request Sep 24, 2023
Open
20 tasks
@joyeecheungjoyeecheung mentioned this pull request Sep 25, 2023
Closed
ruyadorno pushed a commit that referenced this pull request Sep 28, 2023
@nodejs-github-bot@ruyadorno
deps: update undici to 5.24.0
61d18d6
PR-URL: #49559 Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Matthew Aitken <[email protected]>
This was referenced Sep 28, 2023
Closed
Merged
alexfernandez pushed a commit to alexfernandez/node that referenced this pull request Nov 1, 2023
@nodejs-github-bot@alexfernandez
deps: update undici to 5.24.0
de9a202
PR-URL: nodejs#49559 Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Matthew Aitken <[email protected]>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mcollinamcollinamcollina approved these changes

@KhafraDevKhafraDevKhafraDev approved these changes

@ronagronagAwaiting requested review from ronag

Assignees

No one assigned

Labels

commit-queue-squashAdd this label to instruct the Commit Queue to squash all the PR commits into the first one.dependenciesPull requests that update a dependency file.lib / srcIssues and PRs related to general changes in the lib or src directory.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Unexpected error when using signal with reason in fetch

6 participants

@nodejs-github-bot@aduh95@KhafraDev@mcollina@targos