Skip to content

Commit e0684bc

Browse files
committed
Disable Node HTTP request timeout. (#7190)
Also fix user-info returning a 500 when not authenticated. #7183 (cherry picked from commit bc98af9)
1 parent 16f0b8d commit e0684bc

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

api/server.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ process
7070
logger.error('%o',err);
7171
process.exit(1);
7272
}
73-
73+
7474
// Define error-handling middleware last.
7575
// http://expressjs.com/guide/error-handling.html
7676
app.use((err, req, res, next) => {
@@ -82,8 +82,8 @@ process
8282
serverUtils.serverError(err, req, res);
8383
});
8484

85-
app.listen(apiPort, () => {
85+
const server = app.listen(apiPort, () => {
8686
logger.info('Medic API listening on port ' + apiPort);
8787
});
88-
88+
server.setTimeout(0);
8989
})();

api/src/routing.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -388,7 +388,7 @@ app.get('/api/v1/users', users.get);
388388
app.postJson('/api/v1/users', users.create);
389389
app.postJson('/api/v1/users/:username', users.update);
390390
app.delete('/api/v1/users/:username', users.delete);
391-
app.get('/api/v1/users-info', authorization.getUserSettings, users.info);
391+
app.get('/api/v1/users-info', authorization.handleAuthErrors, authorization.getUserSettings, users.info);
392392

393393
app.postJson('/api/v1/places', function(req, res) {
394394
auth

0 commit comments

Comments
 (0)