Skip to content

Conversation

karthikkondapally
Copy link

fixes #1857

  • Clone http.DefaultTransport to inherit future Go defaults
  • If cloning fails, construct and return a new http.Transport
    with standard fallback configuration

Signed-off-by: Karthik Kondapally <kondapally9@gmail.com>
Signed-off-by: Karthik Kondapally <kondapally9@gmail.com>
@karthikkondapally
Copy link
Author

cc @bwplotka

@karthikkondapally
Copy link
Author

karthikkondapally commented Oct 7, 2025

Tansport.clone has issues with Http2 and is mentioned in this golang/go#39302 and even 1.24 has issues golang/go#39302(comment) ,
I will just simply add additional missing variables to old Transport construction.

@karthikkondapally karthikkondapally marked this pull request as draft October 7, 2025 15:33
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.

Why not use the http.DefaultTransport of go sdk as the api.DefaultTransport value
1 participant