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

Update databases.md #4891

Merged
merged 1 commit into from
Sep 27, 2013
Merged

Update databases.md #4891

merged 1 commit into from
Sep 27, 2013

Conversation

denics
Copy link
Contributor

@denics denics commented Aug 25, 2013

run mysql_secure_installation to secure the freshly installed mysql server

run mysql_secure_installation to secure the freshly installed mysql server
@dplarson
Copy link
Contributor

Although I agree this is probably a good idea, can you provide more details/justification for adding this step to the installation guide?

@denics
Copy link
Contributor Author

denics commented Aug 26, 2013

Hi @dplarson,
it is usually a good practice, in production environments, to use the mysql_secure_installation command, to secure the initial installation. The default mysql installation come with the possibility of an empty root password (!), the possibility to connect as root from outside localhost, the possibility to connect as anonymous and a test db. All of these elements are dangerous, and the script has no side effects.

@dplarson
Copy link
Contributor

So mysql_secure_installation:

  1. disables empty root passwords
  2. disables connection as root from outside localhost
  3. disables anonymous connections
  4. removes test database

Sounds good to me. 👍

@denics
Copy link
Contributor Author

denics commented Aug 26, 2013

;)

@axilleas
Copy link
Contributor

👍 That's what we have in the CentOS guide. Maybe add some comments like in CentOS for new users:

Secure MySQL by entering a root password and say "Yes" to all questions

@Razer6
Copy link
Member

Razer6 commented Sep 27, 2013

@jacobvosmaer Looks good

jacobvosmaer added a commit that referenced this pull request Sep 27, 2013
Add mysql_secure_installation to databases.md
@jacobvosmaer jacobvosmaer merged commit 00f33e6 into gitlabhq:6-0-stable Sep 27, 2013
@Razer6
Copy link
Member

Razer6 commented Sep 27, 2013

@jacobvosmaer I did not see that this PR was going to 6-0-stable. I think we should cherry-pick this to master

@jacobvosmaer
Copy link
Contributor

Thanks for spotting that @Razer6. I cherry-picked it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants