enetx / surf / 21510756790

83%

Build Type

push

github

Committed by enetx

Commit Message

feat(core): improve context cancellation support and error handling

Major changes:

Body:
- All read methods (Bytes, String, UTF8) now return g.Result for explicit error handling
- Reworked cancellation mechanism via context: uses SetReadDeadline instead of select in Read()
- Added race condition protection via sync.Once (readOnce, closeOnce, setupOnce)
- Optimized Close(): limited drain to 256KB, skip draining on cancelled context
- Stream() now returns StreamReader with Close() method for automatic resource cleanup

Proxy (connectproxy):
- Renamed DialTLS to DialTLSContext to support cancellation in TLS handshake
- Added context cancellation support in HTTP/1.1 CONNECT via SetDeadline
- Improved HTTP/2 connection reuse: reset connections on errors
- Protected http2.Transport (tr2) with sync.Once against race conditions
- Added SetResolver for custom DNS resolvers
- Improved resource cleanup on HTTP/2 CONNECT errors
- Use errors.Join to combine multiple errors

SOCKS4:
- Implemented DialContext to support context cancellation
- Added deadline setting from context
- Migrated DNS lookup to context-aware resolver.LookupIPAddr
- Pass context to underlying dialer when supported

Other:
- HTTP/2 cleartext (h2c) now uses DialContext instead of Dial
- Debug correctly handles Result types from Body.String()

These changes ensure proper operation cancellation via context,
explicit error handling, and protection against race conditions.

Run Details

126 of 210 new or added lines in 6 files covered. (60.0%)

94 existing lines in 3 files now uncovered.

3095 of 3808 relevant lines covered (81.28%)

73.69 hits per line