Skip to content

Conversation

ip2location
Copy link
Contributor

Added service from https://www.ip2location.io as a new provider.

@jbelien
Copy link
Member

jbelien commented Oct 10, 2023

Hello @ip2location,

We already have a IP2Location provider: https://github.com/geocoder-php/Geocoder/tree/master/src/Provider/IP2Location

What's the difference between IP2Location.com and IP2Location.io ?

@ip2location
Copy link
Contributor Author

Hello @jbelien,

The IP2Location is using an old API interface which we have deprecated. However, we are still supporting it for all existing users.

This PR is for IP2Location.io API which he launched in 2021. The new API is faster and offer free queries.

@ip2location
Copy link
Contributor Author

Hi @jbelien,

Do you need more information from us? Feel free to ask.

@Roensby
Copy link

Roensby commented Aug 28, 2025

This works and would be a useful addition it seems.

@jbelien
Copy link
Member

jbelien commented Oct 4, 2025

Wouldn't it make more sense to create a new major release of the IP2Location provider (version 2.0.0) instead of having to manage 2 different providers for the same provider ?

@ip2location
Copy link
Contributor Author

Hi @jbelien

To ensure legacy support, we will maintain the existing implementation at api.ip2location.com for users still on the old interface. The new interface, IP2Location.io, is hosted on a separate domain.

Therefore, creating a new provider, IP2LocationIO, will prevent confusion while allowing us to support both legacy and new API users.

@jbelien
Copy link
Member

jbelien commented Oct 5, 2025

Okay, I get it ; but is the stack and functionalities behind IP2Location and IP2LocationIo completely different ?
With a new major version you can still easily maintain both in the same provider (with 2 separate branches).

In any case, could you add a paragraph in both READMEs for IP2Location and IP2LocationIo that explain the difference ? Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants