Skip to content
New issue

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

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

Already on GitHub? # to your account

debug print the username and password #17

Closed
wants to merge 1 commit into from
Closed

Conversation

muelli
Copy link

@muelli muelli commented May 27, 2015

This helps #16.

@drybjed
Copy link
Member

drybjed commented Jan 25, 2017

You shouldn't use debug statements in production roles, this leads to sensitive data leaks.

The randomly generated admin password can be found in the ansible/secret/ directory of a given DebOps project directory. Besides, in recent versions, GitLab on first page visit after installation ask you to provide new administrator password to reset it anyway.

@drybjed drybjed closed this Jan 25, 2017
@muelli
Copy link
Author

muelli commented Jan 30, 2017

IIRC the used email address was more difficult to find out for me.

Reg. that patch: Yeah, it makes sense to not merge it. At least not the password. The username is probably less sensitive. In any case, maybe it's useful for somebody still.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants