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

Improve no-root-password with more knowledge from the manpages #739

Merged

Conversation

tianon
Copy link
Member

@tianon tianon commented May 15, 2015

@vbatts care to give this some review? 😄

@vbatts
Copy link
Contributor

vbatts commented May 15, 2015

vbatts@valse ~ (master *) $ for i in $(docker images -q fedora | sort -u ); do echo $i; sh gh739.sh $i ; echo ; done                                                                                                     
717a8c89fa48

93be8052dfb8
error: crypt password detected for root: '$6$DoHwMNPipK/gAVGn$qseLnsvYNGkBBQQDoKfuYWdHR/b6jLMjDwbT32ad97.EvspmbtfqhyQezyYFrqN0Bk/iNTey1upR5i816bGMR1'

d7f0e75cf11f

found it ;-)

@vbatts
Copy link
Contributor

vbatts commented May 15, 2015

so 👍

@tianon
Copy link
Member Author

tianon commented May 15, 2015

😄

Does the logic look sane?

@tianon tianon force-pushed the improved-root-password-test branch from adef829 to d94242d Compare May 15, 2015 19:56
@tianon tianon force-pushed the improved-root-password-test branch from d94242d to a06f6b8 Compare May 15, 2015 19:58
@yosifkit
Copy link
Member

LGTM

yosifkit added a commit that referenced this pull request May 15, 2015
Improve no-root-password with more knowledge from the manpages
@yosifkit yosifkit merged commit 221fefd into docker-library:master May 15, 2015
@yosifkit yosifkit deleted the improved-root-password-test branch May 15, 2015 20:22
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