added c++

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

PRs that need a full CI run.

Issues and PRs related to the net subsystem.

labels

Sep 15, 2023

The introduction of the uv_pipe_bind2 and uv_pipe_connect2 methods in
libuv v1.46.0 changed the behaviour of uv_pipe_bind and uv_pipe_connect.
This broke the ability to connect to abstract domain sockets on linux.
This change ports PipeWrap to use the new uv_pipe_bind2 and
uv_pipe_connect2 methods to restore abstract domain socket support.
Fixes: nodejs#49656
Refs: libuv/libuv#4030

Introduce a new linux-only test for binding to an abstract unix socket
and then making an http request against that socket.
Refs: nodejs#49656

debadree25 added author ready

PRs that have at least one approval, no pending requests for changes, and a CI started.

request-ci

Add this label to start a Jenkins CI on a PR.

labels

Sep 15, 2023

Open

25 tasks

This was referenced

Sep 17, 2023

Open

This was referenced

Sep 19, 2023

Open

ruyadorno pushed a commit that referenced this pull request

Sep 28, 2023

The introduction of the uv_pipe_bind2 and uv_pipe_connect2 methods in
libuv v1.46.0 changed the behaviour of uv_pipe_bind and uv_pipe_connect.
This broke the ability to connect to abstract domain sockets on linux.
This change ports PipeWrap to use the new uv_pipe_bind2 and
uv_pipe_connect2 methods to restore abstract domain socket support.
Fixes: #49656
Refs: libuv/libuv#4030
PR-URL: #49667
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

This was referenced

Sep 28, 2023

Closed

Merged

deokjinkim added a commit to deokjinkim/node that referenced this pull request

Nov 10, 2023

Merged

nodejs-github-bot pushed a commit that referenced this pull request

Nov 12, 2023

We need to handle errors from uv_pipe_connect2()
because return type is `int`.
Fixes: #50652
Refs: #49667
Refs: libuv/libuv#4030
PR-URL: #50657
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: theanarkh <theratliter@gmail.com>

targos pushed a commit that referenced this pull request

Nov 23, 2023

We need to handle errors from uv_pipe_connect2()
because return type is `int`.
Fixes: #50652
Refs: #49667
Refs: libuv/libuv#4030
PR-URL: #50657
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: theanarkh <theratliter@gmail.com>

Closed

Merged

4 tasks

nodejs-github-bot pushed a commit that referenced this pull request

Nov 28, 2023

PR-URL: #50904
Refs: #49667
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Paolo Insogna <paolo@cowtech.it>

targos pushed a commit that referenced this pull request

Dec 4, 2023

PR-URL: #50904
Refs: #49667
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Paolo Insogna <paolo@cowtech.it>

UlisesGascon pushed a commit that referenced this pull request

Dec 11, 2023

We need to handle errors from uv_pipe_connect2()
because return type is `int`.
Fixes: #50652
Refs: #49667
Refs: libuv/libuv#4030
PR-URL: #50657
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: theanarkh <theratliter@gmail.com>

richardlau pushed a commit that referenced this pull request

Mar 25, 2024

PR-URL: #50904
Refs: #49667
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Paolo Insogna <paolo@cowtech.it>