Skip to content
This repository has been archived by the owner on Jul 6, 2018. It is now read-only.

False assumption that a callback exists? #135

Closed
akc42 opened this issue May 18, 2017 · 0 comments
Closed

False assumption that a callback exists? #135

akc42 opened this issue May 18, 2017 · 0 comments

Comments

@akc42
Copy link

akc42 commented May 18, 2017

I was trying to debug the issue #126 so was slowly stepping through in the debugger. So much so that (I presume) something timed out.

I got this error thrown (admittedly before a different error was thrown)

internal/http2/compat.js:412
      cb(new Error('HTTP/2 Stream has been closed'));
      ^

TypeError: cb is not a function
    at Http2ServerResponse.write (internal/http2/compat.js:412:7)
    at ReadStream.ondata (_stream_readable.js:574:20)
    at emitOne (events.js:115:13)
    at ReadStream.emit (events.js:210:7)
    at readableAddChunk (_stream_readable.js:195:18)
    at ReadStream.Readable.push (_stream_readable.js:153:10)
    at onread (fs.js:1977:12)
    at FSReqWrap.wrapper [as oncomplete] (fs.js:650:17)

mcollina added a commit to mcollina/http2 that referenced this issue May 18, 2017
mcollina added a commit to mcollina/http2 that referenced this issue May 18, 2017
mcollina added a commit to mcollina/http2 that referenced this issue May 19, 2017
jasnell pushed a commit that referenced this issue May 19, 2017
Fixes: #135
PR-URL: #137
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
jasnell pushed a commit that referenced this issue May 31, 2017
Fixes: #135
PR-URL: #137
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
jasnell pushed a commit to jasnell/http2-1 that referenced this issue Jun 22, 2017
Fixes: nodejs#135
PR-URL: nodejs#137
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
jasnell pushed a commit to jasnell/http2-1 that referenced this issue Jul 10, 2017
Fixes: nodejs#135
PR-URL: nodejs#137
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
jasnell pushed a commit to jasnell/http2-1 that referenced this issue Jul 14, 2017
Fixes: nodejs#135
PR-URL: nodejs#137
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant