Skip to content

Commit

Permalink
msg.finished is only finished if socket has been detached
Browse files Browse the repository at this point in the history
  • Loading branch information
ronag committed Jun 25, 2019
1 parent 08345db commit ec12f71
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion index.js
Original file line number Diff line number Diff line change
Expand Up @@ -64,10 +64,17 @@ function onFinished (msg, listener) {

function isFinished (msg) {
var socket = msg.socket
var stream = msg.stream

if (stream && typeof stream.closed === 'boolean') {
// Http2ServerRequest
// Http2ServerResponse
return stream.closed
}

if (typeof msg.finished === 'boolean') {
// OutgoingMessage
return Boolean(msg.finished || (socket && !socket.writable))
return Boolean((!socket && msg.finished && msg.outputSize === 0) || (socket && !socket.writable))
}

if (typeof msg.complete === 'boolean') {
Expand Down

0 comments on commit ec12f71

Please sign in to comment.