Here's a few things that I'd like to see from the 8.x-2.0 branch. I'm happy to help implement where they are agreed.

1. A stable release. This helps composer build times, gives us a stable platform to build on and we can get security support.
2. A debugging section on the admin form that allows a user to input an IP and get a result from the current plugin. Screenshot attached.
3. Support for the plugin architecture to getCountry, getCity, supportsCity or similar. Deprecate ->geolocation().

A nice to have would be to:

1. Provide a way to implement auto updating of databases for the enabled plugin from cron and/or drush commands.
2. A hook_requirements() to show which plugin is enabled, and if the plugin supports it, when the data was last updated / out of date.

I really like the D8 architecture and can see this being more generic than using maxminds databases, like the already added CDN plugin.

CommentFileSizeAuthor
manual-check-ip.png78.83 KBnterbogt

Comments

nterbogt created an issue. See original summary.

bojanz’s picture

Welcome aboard!

I've just tagged 8.x-2.0-beta1 with the current state of the codebase. This gives people something to use while we work on RC1.

2. A debugging section on the admin form that allows a user to input an IP and get a result from the current plugin. Screenshot attached.

Sure, why not.

3. Support for the plugin architecture to getCountry, getCity, supportsCity or similar. Deprecate ->geolocation().

Sounds like we need a value object that contains known data, filled by each plugin to the best of its capabilities, right?

A nice to have would be to:

1. Provide a way to implement auto updating of databases for the enabled plugin from cron and/or drush commands.
2. A hook_requirements() to show which plugin is enabled, and if the plugin supports it, when the data was last updated / out of date.

That would be great, let's do it in #2893462: Auto update the maxmind database every month to improve accuracy.

interdruper’s picture

As pointed out in #2893462-3: Auto update the maxmind database every month to improve accuracy, starting December 30, 2019, a MaxMind account is required for downloading the database. Automatic updates would be possible if the license key could be entered in the module config panel.

interdruper’s picture

Another couple of useful features to implement:

  • City geolocation support: right now 8.x only supports Country geolocation.
  • Schema for the database files should be configurable. Now is hardcoded to 'public://'
valic’s picture

Status: Active » Closed (outdated)

Closing this in favor of 3455646 3.1.x plans.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.