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

clients/horizonclient: cant force disable http2 (or support custom http transport) #4154

Closed
JokerCatz opened this issue Dec 30, 2021 · 4 comments
Labels

Comments

@JokerCatz
Copy link

you can do some change like this

https://github.com/grafana/k6/pull/2222/files

the golang old version has http2 deadlock bug , if use goroutine and node rpc or service provider make his gateway to support http2 , the http connection pool possible all stop and never return , even you set all possible timeout configs

@JokerCatz JokerCatz added the bug label Dec 30, 2021
@leighmcculloch
Copy link
Member

leighmcculloch commented Jan 3, 2022

Thanks for opening this issue. Could you clarify which client or which service the issue is occurring on?

Are you running horizon?

Or, are you using one of the clients in the clients/ folder?

@JokerCatz
Copy link
Author

JokerCatz commented Jan 3, 2022

pure golang and yep , I use clients folder

golang/go#32388

@leighmcculloch
Copy link
Member

leighmcculloch commented Jan 3, 2022

I use clients folder

Which client in the clients/ folder are you using?

  • clients/federation
  • clients/horizonclient
  • clients/stellarcore
  • clients/stellartoml

Is it the horizonclient? I will assume so for the rest of this message.

or support custom http transport

The horizonclient.Client type has a field HTTP that you can set with your own http.Client that has its own http.Transport.
https://pkg.go.dev/github.com/stellar/[email protected]/clients/horizonclient#Client

golang/go#32388

This issues appear to have been backported and fixed in Go 1.16.10 (golang/go#48649) and Go 1.17.3 (golang/go#48650). If you can maybe you can upgrade to those versions so that the bug does not affect you.

@leighmcculloch leighmcculloch changed the title clients cant force disable http2 (or support custom http transport) clients: cant force disable http2 (or support custom http transport) Jan 3, 2022
@JokerCatz
Copy link
Author

okay , I checked , many thanx : )

@leighmcculloch leighmcculloch changed the title clients: cant force disable http2 (or support custom http transport) clients/horizonclient: cant force disable http2 (or support custom http transport) Jan 4, 2022
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