sapics/geoip-country

Allow storing the database files in a custom directory

apfelbox opened this issue · 5 comments

apfelbox commented

Hi,

it would be great if it would be allowed to configure a separate directory for storing the database files in.
Our hosting has a readonly-filesystem for everything except specific mounts and I would like to avoid having node_modules completely mutable.

Is that possible?
I could try to help with the implementation.

sapics commented

It looks very good feature!

I create a custom-dir branch for this issue. Could you try the branch?
You could use custom dir with environment GEODATADIR and TMPDATADIR, or CLI parameter --geodatadir=XXXXXX and --tmpdatadir=XXXXXX.

apfelbox commented

The implementation is looking great, I just pushed some small tweaks in #16

  • ❤️1
sapics commented

Thank you for checking and fixing bug! I merge to master branch with your commits!

  • 🚀1
apfelbox commented

Thanks for merging @sapics 🙌

One thing to note is though, that right now, if you install the package with a custom data dir, you need to first update the dbs, as it won't use the bundled ones by default.
That might be worth documenting.

Do you already have an ETA for the next release in mind?

  • 👍1
sapics commented

Thank you for noticing!

I added the documentation about Custom Directory in README. https://github.com/sapics/geoip-country#custom-directory-for-database-files
I hope that the sentense For creating and updating ... explains your concerns. (I am not good at English.)

Currently, this is automatically updated twice a week, so the next update will be made within 48 hours.