Browse Source

Merge pull request #43503 from thaJeztah/optional_kill_signal

Samuel Karp 3 years ago
parent
commit
c7395a4f01
1 changed files with 3 additions and 1 deletions
  1. 3 1
      client/container_kill.go

+ 3 - 1
client/container_kill.go

@@ -8,7 +8,9 @@ import (
 // ContainerKill terminates the container process but does not remove the container from the docker host.
 func (cli *Client) ContainerKill(ctx context.Context, containerID, signal string) error {
 	query := url.Values{}
-	query.Set("signal", signal)
+	if signal != "" {
+		query.Set("signal", signal)
+	}
 
 	resp, err := cli.post(ctx, "/containers/"+containerID+"/kill", query, nil, nil)
 	ensureReaderClosed(resp)