quart icon indicating copy to clipboard operation
quart copied to clipboard

Additional signal arguments

Open pgjones opened this issue 3 years ago • 1 comments

...why the websocket_sent/websocket_received signal don't provide the app object as parameter ? On top of tath, given websocket is a stateful protocol, I would have expected to have websocket_started signal providing a kind of id websocket connection id that is again provided with subsequent websocket_sent/websocket_received/websocket_finished signals 🤔

From https://github.com/pgjones/quart-trio/pull/1

pgjones avatar Jul 07 '22 10:07 pgjones

Both the current app and the current websocket connection are context-local globals. Is there another reason to add them as arguments?

Vivanov98 avatar Sep 02 '22 23:09 Vivanov98