add success status to "safe close" (#191)

bitfinexcom · Aug 23, 2022 · 9f12516 · 9f12516
1 parent 2de38d5
commit 9f12516
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/lib/ws_servers/api/handlers/on_safe_close.js b/lib/ws_servers/api/handlers/on_safe_close.js
@@ -21,7 +21,15 @@ module.exports = async (server, session) => {
  }
  }
 
- await Promise.allSettled(tasks)
+ const results = await Promise.allSettled(tasks)
+ let success = true
 
- send(session, ['app.can_be_closed', true])
+ results.forEach(result => {
+ if (result.status === 'rejected') {
+ success = false
+ console.error('failed to stop', result.reason)
+ }
+ })
+
+ send(session, ['app.can_be_closed', success])
 }
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
  "name": "bfx-hf-server",
- "version": "7.6.0",
+ "version": "7.6.1",
  "description": "HF server bundle",
  "author": "Bitfinex",
  "license": "Apache-2.0",