pires / go-proxyproto / 21589884149

95%

master: 95%

Build Type

Pull #142

github

Committed by clementnuss

Commit Message

fix: Read() data truncation issue

remove the multireader abstraction and modify conn.Read() so that we
start by draining buffered data (from the header parsing), then reading
directly from the underlying connection in a single Read() call

fixes data loss in TLS passthrough scenarios (e.g. ingress-nginx)
when PROXY header + TLS ClientHello exceeds the 256-byte buffer.

Pull Request Pull Request #142: fix: Read() data truncation issue

Run Details

9 of 9 new or added lines in 1 file covered. (100.0%)

14 existing lines in 1 file now uncovered.

952 of 1004 relevant lines covered (94.82%)

50.01 hits per line