Skip to content

Commit 6b56dca

Browse files
authored
Merge pull request #34 from n-e/do-not-forward-sigurg
Do not forward SIGURG to the child process
2 parents 35bb1a2 + 10faccf commit 6b56dca

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

main.go

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -230,8 +230,12 @@ func run(ctx context.Context, provider secrets.Provider, commandSlice []string)
230230
// Goroutine for signals forwarding
231231
go func() {
232232
for sig := range sigs {
233-
// ignore SIGCHLD signals since these are only useful for secrets-init
234-
if sig != syscall.SIGCHLD {
233+
// ignore:
234+
// - SIGCHLD signals, since these are only useful for secrets-init
235+
// - SIGURG signals, since they are used internally by the secrets-init
236+
// go runtime (see https://github.com/golang/go/issues/37942) and are of
237+
// no interest to the child process
238+
if sig != syscall.SIGCHLD && sig != syscall.SIGURG {
235239
// forward signal to the main process and its children
236240
e := syscall.Kill(-cmd.Process.Pid, sig.(syscall.Signal))
237241
if e != nil {

0 commit comments

Comments
 (0)