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

Add support for updating IPv6 #11

Closed
wants to merge 1 commit into from
Closed

Conversation

eljojo
Copy link

@eljojo eljojo commented Oct 7, 2018

Hello,
In this PR I'm trying to add IPv6 support to the image.

The biggest challenge I had was:

  • how to obtain the ipv6, since we cannot just pass ipv6= and expect duckdns to fetch it automatically
  • avoid a second request to duckdns, it seems that ip=&ipv6=[actual_ipv6] ends up only setting the IPv6 leaving the IPv4 untouched.

I'm unsure whether this is the right approach, but I couldn't find a way of making fewer HTTP requests.

I'm very open to suggestions. I'm not really good at bash.

@sparklyballs @aptalca

@LinuxServer-CI
Copy link

Can an admin please whisper to jenkins to test this PR.

@eljojo
Copy link
Author

eljojo commented Oct 7, 2018

As I put this into "production" I kinda just realised it might not make so much sense, since no one really NATs IPv6.

@homerr
Copy link
Member

homerr commented Oct 25, 2018

@eljojo do you want to close this off then or is it worth including? You've done the dirty and put it into production so... how's it work ;)

@homerr homerr self-assigned this Oct 25, 2018
@eljojo
Copy link
Author

eljojo commented Oct 25, 2018

Let's close it. I don't think it makes sense.

@eljojo eljojo closed this Oct 25, 2018
@toobie83
Copy link

My internet connection and I think a lot of other here in germany also changed to DS-Lite - so full ipv6.
Is there any way to add this to duckdns?

@deranjer
Copy link

@toobie83 It would be odd if your provider was issuing you dynamic ipv6 addresses, they should give you a /64 block or something similar that shouldn't change.

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.

None yet

5 participants