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

Allow the idle_timeout parameter to be configured. #5

Merged
merged 1 commit into from
Aug 7, 2016

Conversation

edevil
Copy link
Contributor

@edevil edevil commented Aug 5, 2016

New config parameter to set the Net:Http:Persistent.idle_timeout parameter.
Also sets TCP_KEEPALIVE on for the http socket since it helps for persistent connections.

@roiravhon
Copy link
Contributor

Hey @edevil And thanks for that!
I'll test it locally and merge. looks good.

@roiravhon roiravhon self-assigned this Aug 5, 2016
@roiravhon
Copy link
Contributor

Regard Issue #4.
Looks good and worked locally.

Indeed I have encountered the same issue you reported when trying to send high volume traffic.
When increasing the idle_timeout to be 30 seconds, I was able to send the same bulk without that exception.

Thanks a lot for that fix!
I'll upload the updated version to RubyGems

@roiravhon roiravhon merged commit 1079a1b into logzio:master Aug 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants