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

Missing myip fallback #6

Closed
remkohat opened this issue Feb 21, 2023 · 2 comments
Closed

Missing myip fallback #6

remkohat opened this issue Feb 21, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@remkohat
Copy link

If I'm not mistaken you stated yesterday that when the myip parameter is missing in the uri the script falls back to the ip from where the request originated.
I think yesterday's update might have broken something on that subject.

My Zyxel router at home has a by my ISP customized firmware installed.
I haven't got a clue if a variable for my ip exists and what it might be. Google didn't help one bit on the matter.
image

So I thought that it shouldn't have to be a problem as the script should fall back to the requester's ip.
Unfortunately I was wrong.

I see the expected uri in the logs /nic/update?hostname=<host.ddnsdomain.tld> originating from my home ip.
But nothing is updated in the dns.

When I change ispconfig.tld/nic/update to ispconfig.tld/nic/update?myip=<ip> in the router config, so the uri becomes /nic/update?myip=<ip>&hostname=<host.ddnsdomain.tld>, the dns is updated as expected.

@mhofer117 mhofer117 added the bug Something isn't working label Feb 21, 2023
@mhofer117
Copy link
Owner

You are right the dyndns implementations currently did not check the remote-ip, as it was expected the myip query string is always set.
However this is unrelated to the change yesterday.

Please try the latest release v1.2.4 where this behaviour was added.

@remkohat
Copy link
Author

The update seems to have worked correctly.
Dns is updated with my current ip.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants