breaks heroku about node-geoip

[mariusa]

+1. The data files should be downloaded automatically during build, if not present. We cannot put files manually in module's folder, as it's getting re-downloaded by Heroku.

[bluesmoon]

@mariusa I don't understand. We cannot put the data files into the build for two reasons:

1. The build file becomes too large to upload to npm
2. The data files will be out of date by the time you download the npm, so you'll need to get the new files anyway.

This is why the installation script automatically downloads the data files and converts them during installation.

[mariusa]

This is why the installation script automatically downloads the data files and converts them during installation.

This is great then, thanks.

I am confused why this exists in README:

2 . get the datafiles

[jesper-bylund]

Did you actually get this working? Heroku build is timing out waiting for the database, locally I still need to run the update manually to get any database at all. :/ I'm confused.

[bluesmoon]

I don't know why your heroku build is hanging. As of version 1.1.5, we no longer automatically upgrade the database files on installation, but also as of 1.1.0 we install the built in data files by default so that makes the package itself very large.

[jesper-bylund]

So what's happening right now is:
If I just install the package I get Null as a response. If I try to manually update the DB, Heroku Build times out. That's why I ended up in this thread. :/ Sorry for not having more information. Will dig deeper and report back.

[bluesmoon]

Ok, sorry I can't help, don't know much about what Heroku is doing.

[jesper-bylund]

After a reinstall, it worked. I think I maybe installed an old version? Not sure. Either way, not a bug! Thank you!