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
This fixes#341
The issue was that building os.Process object as &os.Process{pid: 42}, even though it worked in previous Go versions, effectively breaks encapsulation and is not going to work with Go 1.17 at least on Windows. The correct solution is to use os.FindProcess.
Newly introduced function - KillProcessByPid() - should be used in cases where only a process pid is available. Whenever a fully-featured os.Process object is available, its Kill() method is preferred.
See: https://github.com/google/fleetspeak/runs/3537321341?check_suite_focus=true#step:7:138
The issue seems to be caused by the changes Kill() implementation on Windows in Go 1.17: golang/go@105c5b5#diff-6a84f2bc81cd5822adbd431f1f333fa14cbd37e5e88057dd11163ef5319dfd64
The text was updated successfully, but these errors were encountered: