Skip to content

Conversation

norio-nomura
Copy link
Contributor

@norio-nomura norio-nomura commented Sep 9, 2025

Add missing hostSocket support to GRPC port forwarder.

  • Added support for listening on Unix domain sockets
  • Do not set HostPortRange if HostSocket is used.

edit: dropped closing ClosableListeners on exit

@norio-nomura norio-nomura force-pushed the portfwd-support-host-socket branch 2 times, most recently from 277bbce to 141e02e Compare September 9, 2025 07:49
@AkihiroSuda AkihiroSuda added this to the v2.0.0 milestone Sep 9, 2025
@norio-nomura norio-nomura force-pushed the portfwd-support-host-socket branch 2 times, most recently from 64a2088 to 991e95f Compare September 12, 2025 01:27
@norio-nomura
Copy link
Contributor Author

Dropped closing ClosableListeners on exit.
I'll open it as a separate PR.

AkihiroSuda
AkihiroSuda previously approved these changes Sep 19, 2025
Copy link
Member

@AkihiroSuda AkihiroSuda left a comment

Choose a reason for hiding this comment

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

Thanks, can we cover this in test-port-forwarding ?

@norio-nomura
Copy link
Contributor Author

added a test to hack/test-port-forwarding.pl

@norio-nomura norio-nomura force-pushed the portfwd-support-host-socket branch from e7a9d6b to ae4a334 Compare September 21, 2025 06:30
@norio-nomura norio-nomura force-pushed the portfwd-support-host-socket branch from ae4a334 to 490ce12 Compare September 21, 2025 07:03
@norio-nomura norio-nomura force-pushed the portfwd-support-host-socket branch from a315ab7 to bedf7de Compare September 22, 2025 00:45
Because I don't have any Windows environment for debugging.

Signed-off-by: Norio Nomura <[email protected]>
@norio-nomura norio-nomura force-pushed the portfwd-support-host-socket branch from bedf7de to 7ef6404 Compare September 22, 2025 01:13
@norio-nomura
Copy link
Contributor Author

I give up on making hostSocket tests pass on Windows, since I don't have any Windows environments for debugging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants