-
-
Notifications
You must be signed in to change notification settings - Fork 662
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
Automatically update Geoblock files #2663
Conversation
@szaimen This is your implementation, but I think this would be possible. I found that https://www.miyuru.lk/geoiplegacy actually posts the month and year it's been updated, so I thougt that we could scrape that info with curl, and then compare with the current file on the system - and that's what this script does. Please give your opinion. Signed-off-by: Daniel Hansson <[email protected]>
Signed-off-by: Daniel Hansson <[email protected]>
Signed-off-by: Daniel Hansson <[email protected]>
Signed-off-by: Daniel Hansson <[email protected]>
Signed-off-by: Daniel Hansson <[email protected]>
Signed-off-by: Daniel Hansson <[email protected]>
Signed-off-by: Daniel Hansson <[email protected]>
Signed-off-by: Daniel Hansson <[email protected]>
Signed-off-by: Daniel Hansson <[email protected]>
Signed-off-by: Daniel Hansson <[email protected]>
Signed-off-by: Daniel Hansson <[email protected]>
Note to self: check connectivity before proceeding. |
Signed-off-by: Daniel Hansson <[email protected]>
Signed-off-by: Daniel Hansson <[email protected]>
Signed-off-by: Daniel Hansson <[email protected]>
Signed-off-by: Daniel Hansson <[email protected]>
Tested and works, I'll merge this now! |
Hi @enoch85, sorry for coming back to you so late. I was very busy. The reason why I did not implement it like this was this warning on https://www.miyuru.lk/geoiplegacy: |
Valid point. I actually asked the owner of the website myself, and he's fine with it! :)
So the filename has to be |
No, the problem is that the geoip file that is on my system does not match the apache config. Somehow this got messed up after your PR... |
So, what's the output of:
|
I posted the information in the screenshot above already... |
Sorry... I'll have another look and check if we can fix it, probably an easy fix. :) |
Can you try to run this:
|
@szaimen GeoIP is your implementation, but it doesn't automatically fetch latest geofile. To solve that, I made this.
I found that https://www.miyuru.lk/geoiplegacy actually posts the month and year it's been updated, so I thought that we could scrape that info with
curl
, and then compare with the current file on the system - and that's what this change does.Please give your opinion. Maybe there's a smarter way?