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

ddns-scripts: Add option 'myip=no' to Dynu IPv6 update URL #26019

Closed
wants to merge 0 commits into from

Conversation

dmsza
Copy link
Contributor

@dmsza dmsza commented Feb 20, 2025

Maintainer: @feckert
Compile tested: NanoPI R6S, OpenWrt 24.10.0
Run tested: NanoPI R6S, OpenWrt 24.10.0. Confirmed that after this change only IPv6 is updated in DynuDDNS.

Description:
This forces Dynu script to update only the AAAA entry for IPv6 updates. This prevents Dynu from registering an IPv4 address during IPv6 updates. Without this parameter, routers behind a CGNAT with a public IPv6 address would incorrectly register the CGNAT’s IPv4 address, even when updating IPv6 only.

@dmsza
Copy link
Contributor Author

dmsza commented Feb 20, 2025

Missing "Signed-off-by" added to the commit message, fixing the failed test [Test Formalities / Test Formalities (pull_request)].

Other tests failures are not related to this commit (possibly an issue with the test environment?)

@feckert
Copy link
Member

feckert commented Feb 20, 2025

  • You have to update the commit Message. The length constrains are 80 character for each line.
  • Please fix CI/CD this is mostly an incorrect git config on your System. Your first and last name for your Signed-off-by does have to fit with the settings in your git config.

@dmsza dmsza closed this Feb 20, 2025
@dmsza dmsza deleted the fix-dynu-ipv6 branch February 20, 2025 09:40
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