search

sapics / geoip-country

Less memory usage version of geoip-lite by supporting only country lookup.
Other
88 stars 13 forks source link

Update process running for several minutes after update is complete #27

Closed chkp-noamst closed 4 months ago

chkp-noamst commented 5 months ago

Hi all

Thanks for this great repo! I have inserted the updateDb process into the Docker build of my application. After doing so, I noticed that after the update is completed successfully, the process hangs for several minutes before continuing: 

#8 61.09 Fetching new databases from MaxMind...
#8 61.09 Storing files at /usr/src/app/node_modules/geoip-country/data
#8 61.09 Fetching edition GeoLite2-Country-CSV from https://download.maxmind.com/app/geoip_download
#8 61.11 Retrieving GeoLite2-Country-CSV.zip ... DONE
#8 62.30 Extracting GeoLite2-Country-CSV.zip ...Processing Lookup Data (may take a moment) ... DONE
#8 63.46 Processing Data (may take a moment) ...
#8 68.46 Still working (58248) ...
#8 73.46 Still working (142926) ...
#8 78.46 Still working (225284) ...
#8 83.47 Still working (296735) ...
#8 88.46 Still working (361758) ...
#8 93.46 Still working (433216) ... DONE
#8 96.97 Processing Data (may take a moment) ...
#8 102.0 Still working (44718) ...
#8 107.0 Still working (97276) ...
#8 112.0 Still working (146245) ...
#8 [117] Still working (192787) ...
#8 [122] Still working (245601) ... DONE
#8 [126] Successfully Updated Databases from MaxMind.
#8 DONE 462.0s

Notice the 340 additional seconds after the success message.
Thanks in advance Noam

sapics commented 5 months ago

Thank you for fixing the issue!

Edit: I update the version to 4.2.81 and published in npm. Could you try this?

sapics commented 4 months ago

I close this issue as resolved. If you have any issues, feel free to reopen this issue.