Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

server crash in h2+caddy config #105

Closed
bash99 opened this issue Jul 2, 2018 · 4 comments
Closed

server crash in h2+caddy config #105

bash99 opened this issue Jul 2, 2018 · 4 comments
Labels

Comments

@bash99
Copy link

bash99 commented Jul 2, 2018

  • I'm submitting a...
  • [*] bug report
  • What is the current behavior?
    server crashed

  • If the current behavior is a bug, please provide the steps to reproduce
    config with h2+caddy as examples
    use curl download 100mb file from cachefly use proxy, and ctrl+c to stop curl before download complete.

  • What is the expected behavior?
    server close the error connection asap

  • What is the motivation / use case for changing the behavior?

  • Please tell us about your environment:

  • blinksocks version: [3.3.3]
  • Node.js version: [10.x]
  • npm version: [6.x]
  • os: [debian 9]
  • Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix)
    server log
events.js:167
      throw er; // Unhandled 'error' event
      ^

Error [ERR_STREAM_DESTROYED]: Cannot call write after a stream was destroyed
    at doWrite (_stream_writable.js:406:19)
    at clearBuffer (_stream_writable.js:517:5)
    at onwrite (_stream_writable.js:465:7)
    at WriteWrap.afterDoStreamWrite [as oncomplete] (internal/http2/core.js:1449:10)
Emitted 'error' event at:
    at onwriteError (_stream_writable.js:425:12)
    at onwrite (_stream_writable.js:456:5)
    at doWrite (_stream_writable.js:406:11)
    at clearBuffer (_stream_writable.js:517:5)
    at onwrite (_stream_writable.js:465:7)
    at WriteWrap.afterDoStreamWrite [as oncomplete] (internal/http2/core.js:1449:10)
@micooz micooz added the bug label Jul 2, 2018
@micooz
Copy link
Member

micooz commented Jul 2, 2018

Can you install the latest build and check again?

$ npm i -g https://github.com/blinksocks/blinksocks-nightly-releases/raw/master/v3.3.4-20180702071427-d9050ef.tgz

@micooz
Copy link
Member

micooz commented Jul 2, 2018

Have you also experienced a large amount memory usage when download a large file via websocket or http2? This will probably be fixed in the next release, please stay tuned.

@bash99
Copy link
Author

bash99 commented Jul 3, 2018

@micooz yes, last build fix crash problem.
And blinksocks use 167M memory after that download (even cancelled on curl side)

@micooz
Copy link
Member

micooz commented Jul 7, 2018

Fixed in v3.3.4, thanks for your participation.

@micooz micooz closed this as completed Jul 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants