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.

Sign up for GitHub

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

Jump to bottom

try to fix issue #3317, send http response header before sending logs #3695

Closed

C0reFast wants to merge 1 commit into hashicorp:master from C0reFast:master

C0reFast commented Nov 15, 2017

the client freezes because agent didn't send response header, so just call flusher.Flush() to send http header to client before send any logs


          fix issue hashicorp#3317, send http response header before sending logs

9c8cc7f

C0reFast mentioned this pull request

Consul monitor client freezes terminal when no messages are pending #3317

Closed

MichaelSteffens mentioned this pull request

consul monitor doesn't respond to SIGINT/TERM until initial log lines come in #3891

Closed

Member

banks commented Apr 30, 2018

Thanks for this PR - we missed it and ended up merging an identical fix against a separate issue:

Fixed in #3891, released in 1.0.7

banks closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet