Skip to content

Commit

Permalink
Merge pull request #370 from stevengill/327
Browse files Browse the repository at this point in the history
fixed ack in ExpressReceiver firing twice. Issue #327
  • Loading branch information
stevengill authored Jan 13, 2020
2 parents d0dce7b + 6dd53ff commit cc4f098
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions src/ExpressReceiver.ts
Original file line number Diff line number Diff line change
Expand Up @@ -88,8 +88,9 @@ export default class ExpressReceiver extends EventEmitter implements Receiver {
clearTimeout(timer);
timer = undefined;

if (!response) res.send('');
if (typeof response === 'string') {
if (!response) {
res.send('');
} else if (typeof response === 'string') {
res.send(response);
} else {
res.json(response);
Expand Down

0 comments on commit cc4f098

Please sign in to comment.