fix: there should be only one exit event

sysid / sse-starlette

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

504 stars 35 forks source link

fix: there should be only one exit event #57

Closed metakot closed 1 year ago

metakot commented 1 year ago

So AppStatus.should_exit_event got rewrited in listen_for_exit_signal with every new connection being made and old connections remain in connected state when server stops. This fixes that.

sysid commented 1 year ago

Nicely spotted @metakot, thank you !

sysid commented 1 year ago

Problem is: Parametrized tests are failing now with: RuntimeError: <asyncio.locks.Event object at 0x1046a0a30 [unset]> is bound to a different event loop

I fixed it.