We've had problems with the use of nc in the integration tests, for a couple reasons:
The try/finally macros we're using don't seem to run on SIGTERM, so that can leave processes running after glow has been killed.
There are different versions of netcat with incompatible flags on different systems.
We should replace this with something more robust, perhaps in-process network support. I don't really anticipate in the long term that invoking glow with --handhsake will be common.
In GitLab by @isd on Sep 7, 2021, 03:53
We've had problems with the use of
nc
in the integration tests, for a couple reasons:SIGTERM
, so that can leave processes running after glow has been killed.We should replace this with something more robust, perhaps in-process network support. I don't really anticipate in the long term that invoking glow with
--handhsake
will be common.