Spec: Added `reason` to WebSocket Disconnect

django / asgiref

ASGI specification and utilities

https://asgi.readthedocs.io/en/latest/

BSD 3-Clause "New" or "Revised" License

1.46k stars 207 forks source link

Spec: Added `reason` to WebSocket Disconnect #462

Closed frankie567 closed 2 months ago

frankie567 commented 4 months ago

This is a proposal to fix the discussion in #459

Clients can send this reason field when disconnecting. This is allowed as per RFC 6455 and supported in browsers JS API, but previously not supported in ASGI.

I took the liberty to bump the spec version, but I can remove it if you prefer 🙂

frankie567 commented 3 months ago

Anything I can do so we can go forward with this? 🙂

andrewgodwin commented 2 months ago

Thanks for your help on this!