Repository navigation

fix: send a close frame for over-long reasons and invalid codes - #583

Draft

mafredri wants to merge 19 commits into

masterfrom

mafredri/close-reason-truncate

Draft

fix: send a close frame for over-long reasons and invalid codes#583
mafredri wants to merge 19 commits into
masterfrom
mafredri/close-reason-truncate

Commits on Oct 23, 2026
- fix(close): send fallback close frame for invalid codes
  mafredri
  committed
  1223d27
- feat(close): truncate close reasons over 123 bytes
  mafredri
  committed
  5427a1f
- fix(wasm): truncate close reasons over 123 bytes
  mafredri
  committed
  550d8cf
- docs(close): limit the Close guarantee to the first call
  mafredri
  committed
  db23bea
- fix: stop the truncation error from claiming bytes were sent
  mafredri
  committed
  f7d9efa
- docs(ws_js.go): note that invalid UTF-8 reasons can still be rejected
  mafredri
  committed
  fd190bf
- test: cover U+FFFD at the limit and truncation with a write failure
  mafredri
  committed
  052f77c
- fix: replace invalid UTF-8 in close reasons with U+FFFD
  mafredri
  committed
  3c08db6
- docs(ws_js.go): document U+FFFD replacement in the Close reason
  mafredri
  committed
  e9a0c93
- fix: report the replaced reason length in the truncation error
  mafredri
  committed
  f788d53
- fix: put the close failure before the truncation error
  mafredri
  committed
  007027a
- refactor: move wireCloseReason out of errors.go
  mafredri
  committed
  9726c86
- refactor: derive maxCloseReason from maxControlPayload again
  mafredri
  committed
  2bd2b4a
- docs: drop comments that repeat the Close docs
  mafredri
  committed
  7daec2c
- docs: restore the comments that explain the close error handling
  mafredri
  committed
  0b09d99
- fix: say why a truncated reason is longer than the caller's
  mafredri
  committed
  a747f21
- fix: return the truncation error only when Close succeeds
  mafredri
  committed
  b8f74f0
- fix: encode every wireCloseError result in CloseError.bytes
  mafredri
  committed
  ec299b6
- docs(close): drop the Close doc line about non-nil errors
  mafredri
  committed
  ac78959

Commits

Commits on Oct 23, 2026