Cannot download GeoIP.dat.gz over HTTPS

[RigacciOrg]

The default URL for the GeoIP.dat database is broken, it returns a 404 Not Found (October 2024):
http://www.maxmind.com/download/geoip/database/GeoLiteCountry/GeoIP.dat.gz

I tried to configure an alternative URL into downloads.ini, the one I found is HTTPS:
https://mailfud.org/geoip-legacy/GeoIP.dat.gz

Unfortunately it seems that MLDonkey is unable to use HTTPS; after configuring the URL above, I sniffed the traffic toward the target server and I see a TCP connection to TCP port zero instead of 443. Even though I explicitly declare port 443 in the URL, the connection does not happen.

Maybe MLDonkey is not HTTPS capable at all, can be related to issue #41 too.

[raulfg3]

I use Options -> Web Infos to load and appear to work ( need to copy by hand on web_infos folder first time):

[raulfg3]

but not sure if is loaded by URL when reboot or is always the same I copy by hand first time, apparently the date of the file change between reboots

[raulfg3]

anybody can re-test?

[raulfg3]

related: #38

[ogarcia]

@RigacciOrg @raulfg3 the system currently does not work, the only solution is to download the file by hand and update it by hand.

One option to automate it would be to edit the systemd unit that starts mldonkey and put an ExecStartPre that downloads the file when the daemon starts. Another could be to schedule a cron (with systemd or crontab) to do it periodically.

IMHO neither is worthwhile because it is not such a live file that you can't manually update it from time to time.

[raulfg3]

Thanks a lot to test, and for your suggestion. because is only update fewtimes a year, I doi by hand .

Perhaps developer read this and update code .

[carlonluca]

For those of you adventurous enough, I uploaded some patches to fix this problem: https://github.com/carlonluca/mldonkey/tree/feature/curl.

You'll have to build the branch feature/curl from my fork of mldonkey and use the URL https://dl.miyuru.lk/geoip/maxmind/country/maxmind4.dat.gz. I also uploaded an experimental docker image with these patches.

I'm still testing the implementation, but it seems my geoip db was actually updated properly from that URL.