Skip to content

Fixed extra appended "s" in STOP_SERVER_DELAY_COMMAND - #4242

Merged
itzg merged 1 commit into
itzg:masterfrom
harrisonablack:fix/mcServerRunner
Aug 20, 2026
Merged

Fixed extra appended "s" in STOP_SERVER_DELAY_COMMAND#4242
itzg merged 1 commit into
itzg:masterfrom
harrisonablack:fix/mcServerRunner

Conversation

@harrisonablack

Copy link
Copy Markdown
Contributor

Extra appended "s" in STOP_SERVER_DELAY_COMMAND` appends to command message

itzg/minecraft-server:latest

"S" appended to final log, "Server shutting downs"

mc-1  | 2026-08-20T00:10:13.358Z        INFO    mc-server-runner        gracefully stopping server...
mc-1  | 2026-08-20T00:10:13.359Z        INFO    mc-server-runner        Sending shutdown command to Minecraft server
mc-1  | [00:10:13] [RCON Listener #1/INFO]: Thread RCON Client /0:0:0:0:0:0:0:1 started
mc-1  | [00:10:13] [Server thread/INFO]: [Not Secure] [Rcon] Server shutting downs
shutting down

Local build

# Local Build
mc-1  | 2026-08-19T23:50:30.803Z        INFO    mc-server-runner        gracefully stopping server...
mc-1  | 2026-08-19T23:50:30.803Z        INFO    mc-server-runner        Sending shutdown command to Minecraft server
mc-1  | [23:50:30] [RCON Listener #1/INFO]: Thread RCON Client /0:0:0:0:0:0:0:1 started
mc-1  | [23:50:30] [Server thread/INFO]: [Not Secure] [Rcon] Server shutting down
shutting down

refs: #4241

@itzg itzg added the bug label Aug 20, 2026
@itzg itzg linked an issue Aug 20, 2026 that may be closed by this pull request

@itzg itzg left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That was probably a long lurking one. Thanks.

@itzg itzg changed the title fix(mcServerRunner): Extra appended "s" in stop server delay command Fixed extra appended "s" in STOP_SERVER_DELAY_COMMAND Aug 20, 2026
@itzg
itzg merged commit f61f721 into itzg:master Aug 20, 2026
8 checks passed
@harrisonablack
harrisonablack deleted the fix/mcServerRunner branch August 20, 2026 00:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Possible typo in parameter STOP_SERVER_DELAY_COMMAND

2 participants