From 3bbd0d49a91d71d9e56e2662fe1dd69dba5bba36 Mon Sep 17 00:00:00 2001 From: Brian Gough Date: Thu, 12 Mar 2015 10:27:50 +0000 Subject: [PATCH] don't let error events overwrite exit event (not sure if this can happen but it would be bad if it did) --- services/spelling/app/coffee/ASpellWorker.coffee | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/services/spelling/app/coffee/ASpellWorker.coffee b/services/spelling/app/coffee/ASpellWorker.coffee index 79d0e66cdc..0915ff89b2 100644 --- a/services/spelling/app/coffee/ASpellWorker.coffee +++ b/services/spelling/app/coffee/ASpellWorker.coffee @@ -16,12 +16,12 @@ class ASpellWorker logger.log process: @pipe.pid, lang: @language, "aspell worker has exited" metrics.inc "aspellWorker-exit-" + @language @pipe.on 'error', (err) => - @state = 'error' + @state = 'error' unless @state == 'killed' logger.log process: @pipe.pid, error: err, stdout: output, stderr: error, lang: @language, "aspell worker error" if @callback? @callback err, [] @pipe.stdin.on 'error', (err) => - @state = 'error' + @state = 'error' unless @state == 'killed' logger.log process: @pipe.pid, error: err, stdout: output, stderr: error, lang: @language, "aspell worker error on stdin" if @callback? @callback err, []