upgrade node to 8.12.0, upgrade npm packages (express, supertest, webpack, swagger-ui-dist) and fix memory problem #3953
Conversation
…20.2 and reimplement nightscout#3819 (webpack)
|
I tested this out on my server running on a t3.micro with 1GB RAM. Unfortunately, it does not appear to fix the out of memory issue when running webpack. With swapfile off, it locks up the system like it did before. With swapfile on it uses all of the RAM and some swap. |
…" from "^3.18.0" => "^3.19.3".
|
@bassettb I found I missed some of @nielsvanvelzen fixes. Now reapplied them with the Nightscout default formatting ident of 2. Can you please retest and report back if it works (or not). I'm running this branch on Heroku and can't find any problems. |
PieterGit
changed the title
…into 201809_npm_node_update
|
Updated to latest dev. Works on Heroku, but I would like to get confirmation that it now works on t3.micro with 1GB RAM or even in a situation with less RAM (e.g. 512MB). @bassettb could you test that and report back? |
PieterGit
changed the title
…into 201809_npm_node_update
…t Node 10 with Travis - Due to Azure we had several supported npm version 5.6.x, 5.7.x, 5.8.x. Since Node 8.12.0 ships with npm 6.4.x we should stop supporting npm 5.x - Azure currently does not have Node 8.12.x and probably does not work at this moment. Microsoft should stay current - upgrades several packages, most notable mongodb - move file-loader, expose-loader, style-loader to devDependencies - remove webpack-command because it's not longer maintained and has security issues. Moving to webpack-cli - upgrade version to 0.11.0-dev-20181022
resolves (node:24176) DeprecationWarning: current URL string parser is deprecated, and will be removed in a future version. To use the new parser, pass option { useNewUrlParser: true } to MongoClient.connect.
|
Merged to dev. Don't know if this works under Azure which currently still has Node 8.11.1. |
|
I tested it just now and it's still using over 750MB of RAM when running webpack. |
also reimplement #3819 (webpack) without reformatting
Todo: test on Azure and confirm that nodejs 8.12.0 is available there.