Skip to content
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.

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

rs: support ipv6 port-forwarded services #292

Merged
merged 1 commit into from
Aug 21, 2023

Conversation

connor4312
Copy link
Member

Changes proposed:

ipv6-only services are getting more common, especially with a Node 18
change that causes it to listen on ipv6 by default on most enabled
networks. In advance of releasing local forwarding for vscode, I think
we should support this.

This PR supports local ipv6 forwarding by reaching out to both the ipv6
and ipv4 interfaces by default, and using whichever one (if any)
accepts the TCP connection first.

Other Tasks:

  • If you updated the Go SDK did you update the PackageVersion in tunnels.go

ipv6-only services are getting more common, especially with a Node 18
change that causes it to listen on ipv6 by default on most enabled
networks. In advance of releasing local forwarding for vscode, I think
we should support this.

This PR supports local ipv6 forwarding by reaching out to both the ipv6
and ipv4 interfaces by default, and using whichever one (if any)
accepts the TCP connection first.
@connor4312 connor4312 enabled auto-merge (squash) August 18, 2023 20:40
@connor4312 connor4312 merged commit 0f994c6 into main Aug 21, 2023
11 checks passed
@connor4312 connor4312 deleted the connor4312/ipv6-port-forwarding branch August 21, 2023 17:38
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