Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add mc-send-to-console support #228

Merged
merged 2 commits into from
Mar 10, 2022
Merged

Add mc-send-to-console support #228

merged 2 commits into from
Mar 10, 2022

Conversation

itzg
Copy link
Owner

@itzg itzg commented Mar 10, 2022

No description provided.

@itzg itzg merged commit f5c95df into master Mar 10, 2022
@itzg itzg deleted the feat/send-to-console branch March 10, 2022 23:07
itzg added a commit that referenced this pull request Mar 10, 2022
@itzg
Copy link
Owner Author

itzg commented Mar 10, 2022

NOTE this broke the stdin handling of the entrypoint demoter and caused commands via docker attach to report:

WARN[0510] failed to read stdin                          error="write |1: broken pipe"

I should instead enhance entrypoint demoter to create and read from the fifo pipe like mc-server-runner does.

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.

1 participant