glow
glow copied to clipboard
replace use of netcat in the test suite with something less fragile.
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:
- 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:54
mentioned in merge request !168