nix-community / lorri

Your project’s nix-env [maintainer=@Profpatsch,@nyarly]
Apache License 2.0
638 stars 24 forks source link

fix(daemon/server): break out of the stream-events handler #36

Closed Profpatsch closed 3 years ago

Profpatsch commented 3 years ago

If the client disengages, we need to stop the handler immediately, otherwise we keep it running in the for-loop for all times.

This fixes a problem where any call of the lorri internal stream-events --kind snapshot client would drop the connection, and the handler would trace it but not stop itself.

Fixes https://github.com/nix-community/lorri/issues/35

Profpatsch commented 3 years ago

cc @symphorien