Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix #1726 by disabling stale connection checking in AHC5 #1727

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

Merged
merged 1 commit into from
Nov 2, 2021

Conversation

tyge68
Copy link
Contributor

@tyge68 tyge68 commented Oct 28, 2021

As described in #1726 in order to make sure it doesn't fail on stale connection checking, it need to be disabled via the "setValidateAfterInactivity" setup (default is set to 2 seconds in the PoolingHttpClientConnectionManager.

Avoiding checking for stale resolve the issue for me

…alls

Avoiding checking for stale resolve the issue for me
@bsideup
Copy link
Member

bsideup commented Oct 28, 2021

@tyge68 wow, great finding! Thanks. Triggered the CI run, will merge once green 👍

@tyge68
Copy link
Contributor Author

tyge68 commented Nov 2, 2021

@bsideup do you have some info about when it will be merged ? and around when there will be a new release for it ?

@bsideup bsideup changed the title #1726 Fix issue with possible hanging in the isStale(..) calls Fix #1726 by disabling stale connection checking in AHC5 Nov 2, 2021
@bsideup bsideup merged commit c61da29 into docker-java:master Nov 2, 2021
@bsideup
Copy link
Member

bsideup commented Nov 2, 2021

Hi @tyge68,

I just merged the PR, thanks! I will try to merge a couple of other things and do a release relatively soon 👍

@vikinghawk
Copy link

vikinghawk commented Dec 2, 2021

@bsideup any updates on when a release would be available? I believe my team has encountered this same issue a couple times.

@martijndwars
Copy link

I ran into the same issue as @tyge68 described in #1726. A new release that includes this fix would be greatly appreciated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants