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

Run server as minecraft user rather than root #38

Merged
merged 1 commit into from
Dec 1, 2024

Conversation

Jepson2k
Copy link
Contributor

Resolves #37

@TheRemote TheRemote merged commit b4f4e12 into TheRemote:main Dec 1, 2024
5 checks passed
@TheRemote
Copy link
Owner

And this didn't cause any problems? As long as it didn't then I am not opposed to it. Thanks!

@apranger
Copy link

apranger commented Dec 7, 2024

I run this docker on unraid and this update is not playing nicely with the way it manages users & permissions. The script just bails out. Still debugging to see if I can narrow down an error message

@apranger
Copy link

apranger commented Dec 8, 2024

This didn't seem to like the "--restart unless-stopped" option. I pulled that out and at least got the container booting again.

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.

Server runs as root
3 participants