Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GeoIP license change on 30.12.19 #3856

Closed
mimugmail opened this issue Dec 23, 2019 · 31 comments
Closed

GeoIP license change on 30.12.19 #3856

mimugmail opened this issue Dec 23, 2019 · 31 comments
Assignees
Labels
feature Adding new functionality
Milestone

Comments

@mimugmail
Copy link
Member

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md

[X] I have searched the existing issues and I'm convinced that mine is new.

Is your feature request related to a problem? Please describe.

On 30th december the license of GeoIP dabase will change and an account is required to download signatures. Also the download location will change.

https://blog.maxmind.com/2019/12/18/significant-changes-to-accessing-and-using-geolite2-databases/

Describe the solution you'd like

Maybe we have to force all users to create an account and save them somewhere.
In the meantime there won't be any updates

@AdSchellevis AdSchellevis self-assigned this Dec 23, 2019
@AdSchellevis
Copy link
Member

I'm not sure if there are alternatives, need to look into it.

@AdSchellevis
Copy link
Member

pretty inconvenient this, there aren’t a lot of other choices, the link to create an account is currently non-existent and the current link will fail to work in about a week....

@oschwald
Copy link

The blog post has been updated with the new license and sign-up details.

@AdSchellevis
Copy link
Member

@oschwald ok, thanks, I'll take a look.

@AdSchellevis
Copy link
Member

It's a work in progress, you can enter the download location now
image

We might need a bit more feedback and documentation, maybe also document the zip payload so other option might be possible as well.

@mimugmail
Copy link
Member Author

Looks nice, thx. I'll create an account later today

@marjohn56
Copy link
Member

I've added the zip link, should it update as soon as I click save?

@mimugmail
Copy link
Member Author

CSV zip link works great, you'll see the downloads in your statistics at maxmind.
Seems they are collecting every bit of your privacy ...

@marjohn56
Copy link
Member

Yep, it worked overnight. The timestamp is a little ambiguous though, I think that it needs to be made clear that it relates to that date/time that Maxmind updated the data, otherwise I foresee comments and questions arising.

Nice work Ad..

@AdSchellevis
Copy link
Member

I'm likely going to add a bit of documentation about the settings.
We might consider hosting a file for our commercial users ourselves, although we haven't decided about that yet.

@marjohn56
Copy link
Member

Sounds good, If I get chance I'll create a wiki for it, I have a few days off but at present I'm re-purposing my mail server ( no longer using it ) into a Hyper-V system, should be done today hopefully.

@marjohn56
Copy link
Member

Have you noticed that if you don't enter a licence key and just leave it like this:

https://download.maxmind.com/app/geoip_download?edition_id=GeoLite2-Country-CSV&licence_key=My_Licence_key&suffix=zip

You still get the file..

I was just creating a wiki help file and clicked on the link before I put my licence key in and I got the file..

@oschwald
Copy link

After 12/30, the license key will be required to be valid. Right now, an invalid key is treated the same as no key for free GeoLite downloads.

@marjohn56
Copy link
Member

Cool.. OK, well it works so that's all good. I'll remove the comment I added to the wiki.

AdSchellevis added a commit that referenced this issue Dec 29, 2019
* show a message when GeoIP is used but no addresses are found
* support other vendors as well (format should be documented in our docs)
@AdSchellevis
Copy link
Member

Added a small message when using GeoIP and no addresses are found (yet)
image

AdSchellevis added a commit to opnsense/docs that referenced this issue Dec 29, 2019
@AdSchellevis
Copy link
Member

We should probably do a squash merge of the changes into stable/19.7, so we can offer a opnsense-patch statement for people to use this until we have a release available. @fichtner what do you think?

@fichtner
Copy link
Member

fichtner commented Dec 29, 2019 via email

AdSchellevis added a commit that referenced this issue Dec 29, 2019
commit cbd86c180feb40d9088deec78905a355db39da86
Author: Ad Schellevis <ad@opnsense.org>
Date:   Sun Dec 29 14:02:22 2019 +0100

    GeoIP, last bits and pieces for #3856

    * show a message when GeoIP is used but no addresses are found
    * support other vendors as well (format should be documented in our docs)

commit e1e15816ae0be00c37981b50d03d0cd9c4cde20e
Author: Ad Schellevis <ad@opnsense.org>
Date:   Wed Dec 25 16:00:09 2019 +0100

    GeoIP download settings, work in progress for #3856

commit d500ab2
Author: Martin Wasley <martin@queens-park.com>
Date:   Fri Dec 27 13:17:32 2019 +0000

    Update geoIPSettings.xml
    Just a little addition to the help.

commit fbe9586
Author: Ad Schellevis <ad@opnsense.org>
Date:   Sun Dec 29 12:37:56 2019 +0100

    GeoIP, minor help text changes.
@AdSchellevis
Copy link
Member

opnsense-patch 0229cd54

should do the trick.

@andreaslink-de
Copy link

I was exactly following the guide, created an account at maxmind,created a licence key, but always only get "Invalid license key" when trying to download the zip as a first validation test - even after more than 5 mins to get the key activated.
Are you sure this works, when do it right from scratch? I just updated to OPNsense 19.7.9.

@marjohn56
Copy link
Member

a pull request has already been issued. Damn sure I originally copied the text from their site when I created the doc. Either they changed it or I was in spelling correct mode.

@AdSchellevis
Copy link
Member

@marjohn56 my machine at home has the following (which worked and still does):

https://download.maxmind.com/app/geoip_download?edition_id=GeoLite2-Country-CSV&license_key=XXXXX&suffix=zip

@AdSchellevis
Copy link
Member

I'll pull the docs and publish them again, didn't see the typo as well :)

@andreaslink-de
Copy link

Yes, solved it as well, there was an issue with "licence_key" vs. "license_key" and I did not recognize it either. Problem solved with correct spelling :). So updating the documentation will fix it!

@AdSchellevis
Copy link
Member

done https://docs.opnsense.org/manual/how-tos/maxmind_geo_ip.html

@andreaslink-de
Copy link

andreaslink-de commented Jan 9, 2020

BTW: The documentation does not say anything about an automated refresh cycle, so do I need to setup a cron in parallel (like "Update and reload firewall aliases") or how often are the GeoIP details updated aka the CSV downloaded? Who/What is responsible for updating it and where can I verify it was successful?

@marjohn56
Copy link
Member

@ad - is there any reason for the Geo page not having a manual test button?

I know you put it together at very short notice, and very well done, but after 20.1 is released should we take another look at that?

@AdSchellevis
Copy link
Member

@marjohn56 I'm not against a test button, but functionally it's the same as it was now. @andreaslink-de it's updated once a day if used (the same as it was before)

do_update = True
if os.path.isfile('/usr/local/share/GeoIP/alias/NL-IPv4'):
fstat = os.stat('/usr/local/share/GeoIP/alias/NL-IPv4')
if (time.time() - fstat.st_mtime) < (86400 - 90):
do_update = False

It doesn't make sense to update more often, since these lists don't change that often anyway.

fichtner pushed a commit that referenced this issue Jan 10, 2020
@marjohn56
Copy link
Member

No I agree with that, but if you are a first time user or as with the kerfuffle that happened yesterday, it would be useful to be able to test. I think the free version is only updated weakly or weekly. :)

@vistalba
Copy link

I followed the guide exactly today morning (https://docs.opnsense.org/manual/how-tos/maxmind_geo_ip.html) and I can download the zip file successfully from my browser.

If I put the link in the opnsense config and click apply nothing happens. Counter still on "0". Also after a reboot so I don't know if it's working.

Everytime I open the "Aliases" again I get the PopUp Message again:

"In order to use GeoIP, you need to configure a source in the GeoIP settings tab."

Is this normal? If it's just once a day, then it should be mentioned in the Docs or build in a test button to test if it works.
Should this message disappear if a proper link is set?

@andreaslink-de
Copy link

Hej @vistalba I had the same problem and finally "fixed it" by going into my GeoIP Alias (first tab) added one random country, saved it and and clicked on "Apply" on the Alias page at the botton. That moment it was refreshed first time and on second tab you could see the file to be downloaded and geolocations were counted. So it seems, it needs at least one change to refresh it all.

@vistalba
Copy link

@andreaslink-de Thanks! Worked for me too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Adding new functionality
Development

No branches or pull requests

7 participants