django / channels

Developer-friendly asynchrony for Django
https://channels.readthedocs.io
BSD 3-Clause "New" or "Revised" License
6.08k stars 800 forks source link

Updated websocket consumers for ASGI v2.3. #2002

Closed kristjanvalur closed 6 months ago

kristjanvalur commented 1 year ago

This PR adds the "headers" field for websocket.accept messages and the "reason" field for websocket.close, which were added in ASGI spec versions 2.1 and 2.3 respectively.

kristjanvalur commented 1 year ago

I've updated, could you re-run the workflow?

kristjanvalur commented 1 year ago

Oops, sorry about that, everything should now pass, including the linting.

kristjanvalur commented 1 year ago

Would be great to have this merged at some point.

kristjanvalur commented 1 year ago

I did the fixes with github.dev, could not test them, but I´m crossing my fingers.