acrisci / simple-breakpad-server

Simple breakpad crash reports collecting server
https://www.npmjs.com/package/simple-breakpad-server
MIT License
67 stars 29 forks source link

not working with node 10 and later #20

Closed tzvetelin-vassilev closed 5 years ago

tzvetelin-vassilev commented 5 years ago

Please add support for latest node js

Jimbly commented 5 years ago

Looks like the problem is the sqlite3@3 module does not work on Node v10+, however v4 of sqlite3 does, and it doesn't seem like there's any breaking changes between v3 and v4, so installing v4 should probably solve the issue for you: npm i --no-save sqlite3@4.0.3. I haven't had time to fully test, but if that works for you, it's probably safe for us to just update to that sqlite3 version.

tzvetelin-vassilev commented 5 years ago

I was not able to integrate this version of sqlite3. I got new errors when try to install it.

tzvetelin-vassilev commented 5 years ago

4.0.3 not works with node 11, they have such issue, we need to wait little bit to fix that

Jimbly commented 5 years ago

Since sqlite3 requires native builds, which can be tricky on some platforms, it's probably safest to stick to the stable/LTS (even numbered) Node releases - it should work on Node v10.x, but might not consistently on v11.x.

Jimbly commented 5 years ago

Fixed in #22.