fix(http): don't send empty `Sec-WebSocket-Protocol` header

ninjasource / embedded-websocket

A rust websocket library for embedded systems (no_std)

Apache License 2.0

98 stars 28 forks source link

fix(http): don't send empty `Sec-WebSocket-Protocol` header #17

Closed kskalski closed 11 months ago

kskalski commented 11 months ago

In the wild some servers do not even accept request when this header is present, but empty. According to spec this header is optional too, so it should be correct and more compatible to not send it when no subprotocols are specified.