-
-
Notifications
You must be signed in to change notification settings - Fork 5.3k
change geolocation service to freegeoip.app #2503
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
Conversation
I feel the extra setting is not really necessary. If freegeoip.app is secure and works well can we simply switch to it, without the settings? |
Hmm, yes and no. If I could be sure that freegeoip.app was as accurate as the other one, I would agree. The problem is that I just don't know. I can't go for a 72h drive to South America and check its accuracy. ;-) Until we can be sure, we should have the option. If after a while people don't complain, we can remove the old one. But it's up to you. |
@laurent22 Please let me know how to proceed. I'm ok with removing the options or make them not public. The fact is that we don't really know how accurate the new provider is, so my initial thought was to provide users a choice in case they are not happy with it. |
If there's a problem with the new provider, it's still a problem whether it's optional or not. If we're reasonably confident it's better than the previous one, then let's use it (with no option, hidden or not). And if we get any feedback about it in the forum, we can always change it. |
Ok, I'll make the option hidden... At least for now, thus we'll have an easier way to deal with new providers in the future. |
121a648
to
17f02e1
Compare
Done and tested. |
Sorry I should have been clearer. I meant I don't want an option because once an option is added, there's no mechanism to remove it (so everybody will have it, forever, even though it's not really used). So if you could simply hardcode whatever URL is best that would be great. And if it turns out there are problems with it, we can change it later on. |
Ok, will do when I'm home |
removed unnecessary request to `api.ipify.org` closes laurent22#2469
17f02e1
to
1a07954
Compare
Done. |
Perfect, thank you! |
Change geo-location service to
freegeoip.app
(https).removed unnecessary request to
api.ipify.org
closes #2469