Skip to content

add a default statut messsage to exec error to avoid empty line display#12987

Merged
ndeloof merged 1 commit intodocker:mainfrom
glours:fix-exec-new-line-error
Jun 27, 2025
Merged

add a default statut messsage to exec error to avoid empty line display#12987
ndeloof merged 1 commit intodocker:mainfrom
glours:fix-exec-new-line-error

Conversation

@glours
Copy link
Copy Markdown
Contributor

@glours glours commented Jun 27, 2025

What I did
Add the default exit status $exit_code message when docker compose exec fails and the Docker cli returns only the status code without any message.
The status message was the one we had few releases ago

Related issue
fix #12962

(not mandatory) A picture of a cute animal, if possible in relation to what you did
image

@glours glours requested a review from a team as a code owner June 27, 2025 14:16
@glours glours requested a review from ndeloof June 27, 2025 14:16
@glours glours self-assigned this Jun 27, 2025
@ndeloof ndeloof enabled auto-merge (rebase) June 27, 2025 14:22
@glours glours force-pushed the fix-exec-new-line-error branch from 0353339 to 1a8d52b Compare June 27, 2025 18:20
@ndeloof ndeloof merged commit 3553aa2 into docker:main Jun 27, 2025
42 of 51 checks passed
@ndeloof ndeloof deleted the fix-exec-new-line-error branch June 30, 2025 05:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Extra newline char in stderr of docker compose exec if command returns non-zero exit status

2 participants