Skip to content

allow setting the superuser password from env#3

Open
icewind1991 wants to merge 1 commit into
mattikus:masterfrom
icewind1991:superuser-from-env
Open

allow setting the superuser password from env#3
icewind1991 wants to merge 1 commit into
mattikus:masterfrom
icewind1991:superuser-from-env

Conversation

@icewind1991

Copy link
Copy Markdown
docker run -d -e SUPERUSER_PASSWORD=foobar mattikus/murmur

Removes the need to check the logs for the password and makes it easier to automate setting up servers

Comment thread entrypoint.sh
#!/bin/sh

if [ "$SUPERUSER_PASSWORD" ]; then
/opt/murmur/murmur.x86 -supw $SUPERUSER_PASSWORD

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.

We'd probably want to change this to still set -fg -v. Or even better append to a string of additional arguments and add those to line 7 below.

@mattikus

mattikus commented Sep 8, 2016

Copy link
Copy Markdown
Owner

Looks pretty solid, aside from the comment I have. Thanks for the contribution.

@mattikus

mattikus commented Sep 8, 2016

Copy link
Copy Markdown
Owner

Oh, can you also update the README with an example usage?

@gunr2171

gunr2171 commented Jul 9, 2020

Copy link
Copy Markdown

Looking forward to this feature!

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.

3 participants