Skip to content

Commit 46ddcd6

Browse files
authored
Remove connect-multiparty librar - not used by engine or gui (#345)
* Remove connect-multiparty librar - not used by engine or gui * Check req.body for undefined * Make sure body is defined by middleware
1 parent e54e2ae commit 46ddcd6

3 files changed

Lines changed: 4 additions & 182 deletions

File tree

package-lock.json

Lines changed: 0 additions & 179 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,6 @@
5959
"chance": "^1.1.11",
6060
"commander": "^14.0.3",
6161
"compression": "^1.7.4",
62-
"connect-multiparty": "^2.2.0",
6362
"content-disposition": "^1",
6463
"cookie-parser": "^1.4.6",
6564
"cors": "^2.8.5",

src/server/standalone.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ const compression = require('compression');
1717
const cookieParser = require('cookie-parser');
1818
const bodyParser = require('body-parser');
1919
const methodOverride = require('method-override');
20-
const multipart = require('connect-multiparty');
2120
const cors = require('cors');
2221
const Http = require('http');
2322
const ejs = require('ejs');
@@ -175,8 +174,11 @@ class StandAloneServer {
175174
this.__app.use(cookieParser());
176175
this.__app.use(bodyParser.json(gmeConfig.server.bodyParser.json));
177176
this.__app.use(bodyParser.urlencoded({extended: true}));
177+
this.__app.use((req, res, next) => {
178+
req.body = req.body || {};
179+
next();
180+
});
178181
this.__app.use(methodOverride());
179-
this.__app.use(multipart({defer: true})); // required to upload files. (body parser should not be used!)
180182

181183
this.__logger.debug('Collecting valid self-addresses');
182184
const networkIfs = OS.networkInterfaces();

0 commit comments

Comments
 (0)