Преглед на файлове

Fix order of .status().send()

Simon Detheridge преди 6 години
родител
ревизия
6a679023d3
променени са 2 файла, в които са добавени 2 реда и са изтрити 2 реда
  1. 1 1
      services/filestore/app.js
  2. 1 1
      services/filestore/app/js/ExceptionHandler.js

+ 1 - 1
services/filestore/app.js

@@ -143,7 +143,7 @@ app.get('/status', function(req, res) {
     res.send('filestore sharelatex up')
     res.send('filestore sharelatex up')
   } else {
   } else {
     logger.log('app is not ok - shutting down')
     logger.log('app is not ok - shutting down')
-    res.send('server is being shut down').status(500)
+    res.status(500).send('server is being shut down')
   }
   }
 })
 })
 
 

+ 1 - 1
services/filestore/app/js/ExceptionHandler.js

@@ -57,7 +57,7 @@ class ExceptionHandler {
         // request a shutdown to prevent memory leaks
         // request a shutdown to prevent memory leaks
         this.beginShutdown()
         this.beginShutdown()
         if (!res.headerSent) {
         if (!res.headerSent) {
-          res.send('uncaught exception').status(500)
+          res.status(500).send('uncaught exception')
         }
         }
         req = {
         req = {
           body: req.body,
           body: req.body,