Просмотр исходного кода

Merge pull request #17871 from overleaf/jpa-bind-v4-v6

[misc] align the host for binding and requesting in tests

GitOrigin-RevId: 916521c56928329ff2cbf2817f3a0a27aeaf8c3d
Jakob Ackermann 2 лет назад
Родитель
Сommit
a6fb983afc

+ 2 - 2
services/clsi/app.js

@@ -368,8 +368,8 @@ loadHttpServer.post('/state/maint', function (req, res, next) {
   res.sendStatus(204)
   res.sendStatus(204)
 })
 })
 
 
-const port = Settings.internal?.clsi?.port || 3013
-const host = Settings.internal?.clsi?.host || 'localhost'
+const port = Settings.internal.clsi.port
+const host = Settings.internal.clsi.host
 
 
 const loadTcpPort = Settings.internal.load_balancer_agent.load_port
 const loadTcpPort = Settings.internal.load_balancer_agent.load_port
 const loadHttpPort = Settings.internal.load_balancer_agent.local_port
 const loadHttpPort = Settings.internal.load_balancer_agent.local_port

+ 2 - 10
services/clsi/test/acceptance/js/helpers/ClsiApp.js

@@ -30,11 +30,8 @@ module.exports = {
       this.initing = true
       this.initing = true
       this.callbacks.push(callback)
       this.callbacks.push(callback)
       return app.listen(
       return app.listen(
-        __guard__(
-          Settings.internal != null ? Settings.internal.clsi : undefined,
-          x => x.port
-        ),
-        'localhost',
+        Settings.internal.clsi.port,
+        Settings.internal.clsi.host,
         error => {
         error => {
           if (error != null) {
           if (error != null) {
             throw error
             throw error
@@ -54,8 +51,3 @@ module.exports = {
     }
     }
   },
   },
 }
 }
-function __guard__(value, transform) {
-  return typeof value !== 'undefined' && value !== null
-    ? transform(value)
-    : undefined
-}

+ 1 - 1
services/web/test/acceptance/src/helpers/InitApp.js

@@ -63,7 +63,7 @@ let server
 
 
 before('start main app', function (done) {
 before('start main app', function (done) {
   addSessionEndpoint(app)
   addSessionEndpoint(app)
-  server = App.listen(23000, 'localhost', done)
+  server = App.listen(23000, '127.0.0.1', done)
 })
 })
 
 
 before('start queue workers', function () {
 before('start queue workers', function () {