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

fix(http2_adapter):request by http2 without network once,the same domain request will never success. #1896

Merged
merged 4 commits into from
Jul 14, 2023
Merged

Conversation

sunhapper
Copy link
Contributor

@sunhapper sunhapper commented Jul 12, 2023

New Pull Request Checklist

  • I have read the Documentation
  • I have searched for a similar pull request in the project and found none
  • I have updated this branch with the latest main branch to avoid conflicts (via merge from master or rebase)
  • I have added the required tests to prove the fix/feature I'm adding
  • I have updated the documentation (if necessary)
  • I have run the tests without failures
  • I have updated the CHANGELOG.md in the corresponding package

Additional context and info (if any)

_ConnectionManager._connect may throw SocketException when call await initFuture,if it occurs, even if the network is restored, subsequent requests with the same domain will still obtain the cached initFuture, and it will still fail

@sunhapper sunhapper requested a review from a team as a code owner July 12, 2023 14:54
@sunhapper sunhapper requested a review from AlexV525 July 14, 2023 16:54
@AlexV525 AlexV525 merged commit 063486e into cfug:main Jul 14, 2023
31 checks passed
@sunhapper sunhapper deleted the fix_http2_block branch July 17, 2023 02:14
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