phpcyy opened a new pull request, #3165:
URL: https://github.com/apache/dubbo-go/pull/3165
This fixes a bug where the Triple client's HTTP/2 transport wasn't
respecting context cancellation and timeouts. The non-TLS path was using
net.Dial instead of DialContext, and the TLS path wasn't explicitly using a
context-aware dialer.
Changes:
- Use (&net.Dialer{}).DialContext for non-TLS HTTP/2 connections
- Use (&tls.Dialer{Config: tlsConfig}).DialContext for TLS connections
- Add test cases to verify timeout behavior with both TLS and non-TLS
The test uses a black-hole address (RFC 5737) to ensure connections time out
as expected without relying on external network resources.
### Description
Fixes # (issue)
### Checklist
- [x] I confirm the target branch is `develop`
- [x] Code has passed local testing
- [x] I have added tests that prove my fix is effective or that my feature
works
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
