Skip to content

client: ContainerKill(): don't send signal query-param if none was set#43503

Merged
samuelkarp merged 1 commit intomoby:masterfrom
thaJeztah:optional_kill_signal
Apr 20, 2022
Merged

client: ContainerKill(): don't send signal query-param if none was set#43503
samuelkarp merged 1 commit intomoby:masterfrom
thaJeztah:optional_kill_signal

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

Just a small clean-up (there's more endpoints to do this for, but I was working on changes in this area on the CLI when I noticed we were setting this query-parameter unconditionally.

- A picture of a cute animal (not mandatory but encouraged)

Just a small clean-up (there's more endpoints to do this for, but
I was working on changes in this area on the CLI when I noticed we
were setting this query-parameter unconditionally.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@samuelkarp samuelkarp merged commit c7395a4 into moby:master Apr 20, 2022
@thaJeztah thaJeztah deleted the optional_kill_signal branch April 20, 2022 10:12
@thaJeztah thaJeztah added this to the 22.04.0 milestone Apr 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants