You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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.
The text was updated successfully, but these errors were encountered:
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.The text was updated successfully, but these errors were encountered: