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

get rid of the message "No directory, logging in with HOME=/" #182

Merged
merged 1 commit into from
Aug 28, 2013
Merged

get rid of the message "No directory, logging in with HOME=/" #182

merged 1 commit into from
Aug 28, 2013

Conversation

JohannesM
Copy link
Contributor

While performing operations with the user mineraft a message appears that the home directory is not available.

While performing operations with the user mineraft a message appears that the home directory is not available.
zachlatta added a commit that referenced this pull request Aug 28, 2013
get rid of the message "No directory, logging in with HOME=/"
@zachlatta zachlatta merged commit 4c8a60f into msmhq:master Aug 28, 2013
@IronTetsubo
Copy link
Contributor

Well, good on you for fixing this (yay! Msm is active once again!), but I think you're fixing the wrong problem. This is also reported in issue #156 and in that thread I have an alternate approach. Namely, to set the msm user to not have a shell at all. This is best practice for accounts that are used to run external services (see most system services on most distros), as it eliminates a whole category of security issues by making it more difficult for any vulnerabilities in minecraft (or java) to let an attacker get to a shell. I can share more references if it's interesting. in the other issue, you can see a reference to my commit that fixes it by eliminating the interactive shell: IronTetsubo@7504ee9

Either way, you should also close out issue #156

@JohannesM JohannesM deleted the patch-1 branch August 29, 2013 09:12
@zachlatta
Copy link
Contributor

Ideally we'd have some sort of switch for this. Although it's not as secure, logging in at the minecraft user is the only way to access the server's console through screen. Once we set it up to allow another user to access the screen session, we'll take away minecraft's shell and remove its home directory. Any comments on this?

@zachlatta zachlatta mentioned this pull request Sep 2, 2013
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.

None yet

3 participants