search

ytorbyk / magento2-geo-ip2

Magento 2 extension: Integrates MaxMind GeoIP2 DB into Magento 2.
MIT License
17 stars 7 forks source link

Credential error with geoip2 v2.9.0 #5

Open mathieuimbert opened 6 years ago

mathieuimbert commented 6 years ago

In the newest geoip2 version v2.9.0, they now use "account" instead of "user".

It means that the client factory should include both userId and accountId in order to support 2.9.0 and the previous versions.

https://github.com/ytorbyk/magento2-geo-ip2/blob/master/Model/WebService/ClientFactory.php#L41

I am not sure if this repository is still maintained, I'll be happy to create a PR is someone can take a look.

ytorbyk commented 6 years ago

@mathieuimbert thank you for pointing, I'll take a look soon.