8000 feat(publicip): add Cloudflare support by jjlin · Pull Request #2502 · qdm12/gluetun · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat(publicip): add Cloudflare support #2502

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

Merged
merged 1 commit into from
Oct 6, 2024
Merged

Conversation

jjlin
Copy link
Contributor
@jjlin jjlin commented Oct 1, 2024

Note that the Cloudflare API does not seem to support returning info for arbitrary IP addresses (only the caller's public IP), so it can't be used with calls to FetchMultiInfo, but since this is only used in the context of a couple of VPN providers currently, it may not be a significant limitation.

Copy link
Owner
@qdm12 qdm12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, still a useful addition I think! As long as it's not the default, it should work fine 😉

Note that the Cloudflare API does not seem to support returning info
for arbitrary IP addresses (only the caller's public IP), so it can't
be used with calls to FetchMultiInfo, but since this is only used in
the context of a couple of VPN providers currently, it may not be a
significant limitation.
@qdm12 qdm12 merged commit cbdd1a9 into qdm12:master Oct 6, 2024
5 checks passed
@qdm12
Copy link
Owner
qdm12 commented Oct 6, 2024

Awesome, thanks!! 💯

@jjlin jjlin deleted the cloudflare branch October 7, 2024 05:46
CMarcJoubert pushed a commit to FideresDev/gluetun that referenced this pull request Jan 14, 2025
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
0