pulkitjalan / ip-geolocation

IP Geolocation Wrapper with Laravel Support
MIT License
85 stars 44 forks source link

Update FREEGEOIP driver to IPSTACK #28

Closed alberto1el closed 6 years ago

alberto1el commented 6 years ago

FREEGEOIP has been deprecated apparently and moved to ipstack. This driver only needs to include a key that you can get for free at the ipstack website and slightly different endpoint. All needed is this new code and to update the keys: GEOIP_DRIVER=freegeoip GEOIP_FREEGEOIP_URL=api.ipstack.com GEOIP_FREEGEOIP_SECURE=false GEOIP_RANDOM=false GEOIP_KEY=##YOUR_IPSTACK_KEY##

pulkitjalan commented 6 years ago

Thanks for this, needs a bit more work with the readme, plus i think the env should probs change, might make this a major release instead. Ill look more into it. Thanks.

alberto1el commented 6 years ago

Thanks for this, needs a bit more work with the readme, plus i think the env should probs change, might make this a major release instead. Ill look more into it. Thanks.

I agree since apparently GEOIP is no longer the name it could be changed to IPSTACK If that is the direction to go I can try and fix that. Should the driver class name also change to IPSTACK? thank you for your response and for the package I use it constantly.