pires / go-proxyproto / 21628228770

95%

master: 95%

Build Type

Pull #142

github

Committed by pires

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%)

13 existing lines in 1 file now uncovered.

963 of 1013 relevant lines covered (95.06%)

51.12 hits per line