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

Is IPv6 supported? #78

Closed
cheerfulstoic opened this issue Jun 27, 2023 · 4 comments · Fixed by #79
Closed

Is IPv6 supported? #78

cheerfulstoic opened this issue Jun 27, 2023 · 4 comments · Fixed by #79

Comments

@cheerfulstoic
Copy link
Contributor

It seems like the opts given to :gen_udp.open/2 maybe needs to have :inet6 given to enable support for IPv6. If so, and if I'm reading the UDP module right, that's not currently possible.

Does that all seem right to you? I could potentially make a PR for this, but I wanted to sanity check first 😄

@cheerfulstoic
Copy link
Contributor Author

Some background on what I'm trying to do, in case it helps: https://community.fly.io/t/using-wireguard-to-connect-to-service-on-my-laptop/13788/3

@cheerfulstoic
Copy link
Contributor Author

I just tried a patched version which hardcodes :inet6 as an option, but that didn't seem to help anything. Maybe it's a network config issue... 🤔 Happy to have advice if you have any 😄

@mbuffa
Copy link

mbuffa commented Jun 27, 2023

Hello there!

Just trying to configure telemetry on fly.io myself, and I think you might be up to something.

In my case, I got a DogStatsD agent running in a container as a dedicated Fly.io app. Fly has internal DNS records and digging the DNS entry on my app works (https://fly.io/docs/reference/private-networking/).

However, calling :inet.gethostbyname("my-app-name.internal") doesn't, it's used by the library, and it has a inet6 option.

@cheerfulstoic
Copy link
Contributor Author

Ok, after a bunch of poking I think that adding the :inet6 option does help. I think that my fork changes weren't going up to fly.io before 😅

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

Successfully merging a pull request may close this issue.

2 participants