Skip to content

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

Merged
merged 1 commit into from
Feb 27, 2020

Conversation

tessus
Copy link
Collaborator

@tessus tessus commented Feb 14, 2020

Change geo-location service to freegeoip.app (https).

removed unnecessary request to api.ipify.org

closes #2469

@tessus tessus added enhancement Feature requests and code enhancements desktop All desktop platforms cli CLI app specific issue PR-tested labels Feb 14, 2020
@laurent22
Copy link
Owner

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?

@tessus
Copy link
Collaborator Author

tessus commented Feb 21, 2020

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.

@tessus
Copy link
Collaborator Author

tessus commented Feb 24, 2020

@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.

@laurent22
Copy link
Owner

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.

@tessus
Copy link
Collaborator Author

tessus commented Feb 24, 2020

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.
I can always delete the option code later.

@tessus tessus changed the title option to choose geolocation service change geolocation service to freegeoip.app Feb 24, 2020
@tessus
Copy link
Collaborator Author

tessus commented Feb 24, 2020

Done and tested.

@laurent22
Copy link
Owner

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.

@tessus
Copy link
Collaborator Author

tessus commented Feb 26, 2020

Ok, will do when I'm home

removed unnecessary request to `api.ipify.org`

closes laurent22#2469
@tessus
Copy link
Collaborator Author

tessus commented Feb 27, 2020

Done.

@laurent22
Copy link
Owner

Perfect, thank you!

@laurent22 laurent22 merged commit a576ad2 into laurent22:master Feb 27, 2020
@tessus tessus deleted the fix/geo-location branch March 12, 2020 23:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cli CLI app specific issue desktop All desktop platforms enhancement Feature requests and code enhancements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

find a good geolocation service (https)
2 participants