-
-
Notifications
You must be signed in to change notification settings - Fork 367
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
Feature request: retry logic for fetching the public ip information #2325
Comments
@qdm12 is more or less the only maintainer of this project and works on it in his free time.
|
Hi there! Thanks for the detailed issue.
Why? 🤔 I'm trying to understand if this is solvable in some cleaner way, than wrapping the public ip information fetch in some retry logic. The public ip fetching is also triggered once the vpn is configured, so it should work, unless the VPN is not working at all. The commit you mention is indeed likely the source of your problem, especially since it 'fixes'
|
So my configuration (docker-compose.yml below) involves blocky for DNS and this works quite well. However, there are times where gluetun has made a connection yet blocky is not yet available for resolution. I've tried variations using Nonetheless, the result is that gluetun will report Would it be possible to retry this somehow that doesn't assume the vpn is failed (as in the commit mentioned above)? docker-compose.yml
|
Closed issues are NOT monitored, so commenting here is likely to be not seen. This is an automated comment setup because @qdm12 is the sole maintainer of this project |
a54be9a adds a retry mechanism only when |
Just writing this last comment made me realize there is a better solution, done in 5a09cb7:
|
Is this urgent?
No
Host OS
Ubuntu 22.04.4 LTS
CPU arch
x86_64
VPN service provider
Custom
What are you using to run the container
docker-compose
What is the version of Gluetun
Running version latest built on 2024-06-17T22:37:52.988Z (commit 93ed87d)
What's the problem 🤔
The publicip endpoint is no longer functional:
I discovered this just recently. I have a control wrapper/application which does a regular automated request against that endpoint and stores the results.
I'm near certain, this is due to this commit: 4218dba
Again, I wish my Go skills and knowledge were better so that I might be able to offer an option.
With my configuration I'm running DNS (blocky) with gluetun (
network_mode: "service:gluetun"
) so I don't expect the initial dns request to succeed. However, it would be nice if there were a latter request which would make the public address available.I did a quick test against
gluetun:v3
and found this was not the case with this release. I also applied the optionalPUBLICIP_API=ip2location
with bothgluetun:v3
andgluetun:latest
and found the issue consistent only with the more recent release.Share your logs (at least 10 lines)
The text was updated successfully, but these errors were encountered: