Skip to content

fix: do not set the host address#217

Merged
sbruens merged 1 commit into
masterfrom
sbruens/ipv4
Oct 2, 2024
Merged

fix: do not set the host address#217
sbruens merged 1 commit into
masterfrom
sbruens/ipv4

Conversation

@sbruens

@sbruens sbruens commented Sep 30, 2024

Copy link
Copy Markdown

This was a regression introduced in #192, which was rolled out in outline-server v1.10.0 released last week and caused issues for users using machines with IPv6 disabled: OutlineFoundation/outline-server#1601.

I'm unable to reproduce the issue itself when IPv6 is disabled on my corp or DigitalOcean machines, but that may be specific to those machines.

@sbruens sbruens requested a review from a team as a code owner September 30, 2024 22:32

@jyyi1 jyyi1 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM. But we need to verify whether it will fix the issue. Probably we can also have some configuration that let the user explicitly specify the IP stack (v4 or v6).

@sbruens

sbruens commented Oct 1, 2024

Copy link
Copy Markdown
Author

LGTM. But we need to verify whether it will fix the issue.

Agreed, though I'm still unable to reproduce this issue, on a host machine or in a Docker container. I had wanted to replicate this in the outline-server integration test, but to no avail. I propose merging this change and, as you suggested offline, creating a new canary release that we could ask affected folks to test.

Probably we can also have some configuration that let the user explicitly specify the IP stack (v4 or v6).

The new service config allows this now at the outline-ss-server layer; the new configuration allows a full address specification, instead of just a port:

https://github.com/Jigsaw-Code/outline-ss-server/blob/cb5965fd1a6982b641c640067b55bcdb1ccca8eb/cmd/outline-ss-server/config_example.yml#L20

But we don't yet support this at the outline-server/Manager layer. I agree it would be nice to explicitly specify the stack there, but I'm also not sure it's totally necessary.

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