Use HTTP instead of FTP

Moving this to 8.x-1.x.

I'm generally opposed to making the Internet === HTTP, but I'm not against adding HTTP as an *option* for those of you who want or need it.

I have added an option to use HTTP connection. Here is a patch

not sure if related, need to check

I re-ran the test and it worked the second time - the failure may have been a testbot problem.

This proved really useful for me because my server's don't have access via FTP to the outside world. Once I implemented the patch, I was able to update the databases again. I implemented this patch against the latest dev version. Once I get back from Christmas vacation, I will try to update the patch so it is up to date with the current version.

I made minor changes to some comments / text strings in #5. I'm happy with this patch, but I think there needs to be at least one test of the downloading using HTTP (actually, the patch uses HTTPS if the option is set ...). That way we will know if this feature ever breaks because of the ARIN server or because of other alterations to the ip2country module.

Forgot the patch ...

Oh, and since we're adding a config variable we need a hook_update_N() to ensure existing sites get the new value, and we need a update test for this hook as well.

New patch attached with the hook and the update test. Still needs a download test like I mentioned in #10.

Splitting off the Drupal 7 issue into #3166385: Use HTTPS instead of FTP (D7 version) to not mix up patches here.

Moving to current development branch.

Rerolled #12 for 2.0.x-dev

Converted #16 to a MR so that the automated testing can test the changes.
I had to make quite a few changes to re-roll it for 2.1.x.

Attached patch from MR.

Merge request updated. Uploading stable patch file here for composer patches.

I think the tests are still missing here.

This is obsolete now, since FTP access was discontinued. Should be closed as outdated

This patch fixes it: https://www.drupal.org/project/ip2country/issues/3532194