go-pkgz / rest / 32198708772

97%

Build Type

push

github

Committed by umputun

Commit Message

fix: send the 101 status line before the connection is handed over

WriteHeader only committed when a Content-Type was already set, so the plain
upgrade sequence, 101 with no content type followed by Hijack, recorded the
status and never forwarded it: Hijack set hijacked and close returned at that
guard, leaving the client with the raw protocol bytes and no status line where
net/http would have written one.

101 now commits immediately, which it can do because it carries no body to sniff
and is already excluded from compression. The regression test omits Content-Type
and reads the raw socket, so it fails without the fix.

Reported by umputun on #51.

Coverage Stats

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

14 existing lines in 2 files now uncovered.

1647 of 1699 relevant lines covered (96.94%)

47.36 hits per line