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

Failed to detect the IPv4 address #486

Closed
ShlomiD83 opened this issue May 18, 2023 · 10 comments · Fixed by #487
Closed

Failed to detect the IPv4 address #486

ShlomiD83 opened this issue May 18, 2023 · 10 comments · Fixed by #487

Comments

@ShlomiD83
Copy link

😞 Failed to send HTTP(S) request to "https://cloudflare-dns.com/cdn-cgi/trace": context deadline exceeded
😞 Failed to detect the IPv4 address

Hi,
Getting this error when using the recent Edge image.
This doesn't happen with the Latest image.

Please assist.

@favonia
Copy link
Owner

favonia commented May 18, 2023

Thank you for the report. Sorry about the trouble. I am experimenting with some unusual mechanism (#484) to detect IPv4 and IPv6 addresses without using hardcoded IP addresses. I guess it doesn't work. :-( Could you possibly share your configuration (but please erase your DOMAIN and API_TOKEN.

@favonia favonia linked a pull request May 18, 2023 that will close this issue
@favonia
Copy link
Owner

favonia commented May 18, 2023

@ShlomiD83 Apologies again for my insufficient testing before merging it into the main branch. I will revert the merge soon. On the other hand, may I understand why you are using the edge tag? Is there a feature you are waiting for that was not published yet?

@favonia favonia reopened this May 18, 2023
@ShlomiD83
Copy link
Author

ShlomiD83 commented May 18, 2023

version: "3"
services:
cloudflare-ddns:
image: favonia/cloudflare-ddns:edge
container_name: cloudflare-ddns
restart: always
cap_add:
- SETUID
- SETGID
cap_drop:
- all
security_opt:
- no-new-privileges:true
read_only: true
network_mode: host
environment:
- CF_API_TOKEN=SomeRandomToken
- DOMAINS=example.com
- PROXIED=true
- PUID=1000
- PGID=1000
- TZ=Asia/Jerusalem
- UPDATE_CRON=@every 5m
- IP6_PROVIDER=none
- DETECTION_TIMEOUT=15s
- QUIET=TRUE

@ShlomiD83
Copy link
Author

@ShlomiD83 Apologies again for my insufficient testing before merging it into the main branch. I will revert the merge soon. On the other hand, may I understand why you are using the edge tag? Is there a feature you are waiting for that was not published yet?

It's stayed that way ever since the Quiet Mode message, never changed it back to the latest tag.

@favonia
Copy link
Owner

favonia commented May 18, 2023

@ShlomiD83 Thank you. Could you confirm that the current edge tag works? If so, maybe we can close this ticket for now. Thank you again for the reporting the bug.

@ShlomiD83
Copy link
Author

@favonia current edge works as expected.
Issue can be closed.
Thank you for your hard work!

@favonia
Copy link
Owner

favonia commented Nov 4, 2023

@ShlomiD83 I forgot to say that it's recommended to use the latest tag (or a fixed version tag) because I'll be experimenting with new code with the edge tag. Thanks!

@ShlomiD83
Copy link
Author

Thank you @favonia .
I've switched back a while ago to the latest tagged image.

@favonia
Copy link
Owner

favonia commented Jul 25, 2024

@ShlomiD83 This is a head-up---recent versions (1.13.0+) will benefit from an update to your configuration, and I am on the mission to eliminate the old template from this universe. In case you or anyone is confused by how to update the configuration, here's the tl;dr:

  1. Remove cap_add: ... completely.
  2. Add user: "1000:1000" (or other IDs you want to use).
  3. Remove PUID=... and PGID=... (optional, but why not).

For more information, please see the CHANGELOG and README.

PS: the new template works for older versions of the updater as well!

@ShlomiD83
Copy link
Author

@ShlomiD83 This is a head-up---recent versions (1.13.0+) will benefit from an update to your configuration, and I am on the mission to eliminate the old template from this universe. In case you or anyone is confused by how to update the configuration, here's the tl;dr:

  1. Remove cap_add: ... completely.
  2. Add user: "1000:1000" (or other IDs you want to use).
  3. Remove PUID=... and PGID=... (optional, but why not).

For more information, please see the CHANGELOG and README.

PS: the new template works for older versions of the updater as well!

Thanks @favonia, I've made the changes.

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 a pull request may close this issue.

2 participants