Skip to content

Commit a3742e4

Browse files
committed
fix: getStatus should reject promise
1. reject promise when getStatus failed 2. change to use promise-way call getStatus in debug message Signed-off-by: Raccoon <[email protected]>
1 parent 54ab0a0 commit a3742e4

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

lib/realtime/realtime.js

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -298,10 +298,12 @@ function getStatus () {
298298
}
299299
})
300300
.catch(function (err) {
301-
return logger.error('count user failed: ' + err)
301+
logger.error('count user failed: ' + err)
302+
return Promise.reject(new Error('count user failed: ' + err))
302303
})
303304
}).catch(function (err) {
304-
return logger.error('count note failed: ' + err)
305+
logger.error('count note failed: ' + err)
306+
return Promise.reject(new Error('count note failed: ' + err))
305307
})
306308
}
307309

@@ -772,8 +774,7 @@ function queueForConnect (socket) {
772774
const noteId = socket.noteId
773775
logger.info('SERVER connected a client to [' + noteId + ']:')
774776
logger.info(JSON.stringify(user))
775-
// logger.info(notes);
776-
getStatus(function (data) {
777+
getStatus().then(function (data) {
777778
logger.info(JSON.stringify(data))
778779
})
779780
}

0 commit comments

Comments
 (0)