Typo in VirtualEnv Installation documentation

VERIFIED FIXED

Status

VERIFIED FIXED
3 years ago
3 years ago

People

(Reporter: shine, Assigned: tasos)

Tracking

2016-1.1

Details

(Whiteboard: [profile-edit-refactor])

(Reporter)

Description

3 years ago
Step 6 of http://mozillians.readthedocs.org/en/latest/installation/installation-virtualenv.html#mysql-setup says :

Install timezone info tables in mysql:
(venv)$ mysql_tzinfo_to_sql /usr/share/zoneinfo/ | mysql -uroot -proot mysql

There should be a space in between the options -u and -p. It is not a good idea (security wise) to input the root password in plain text to a command. Instead the password option should be left open with just the -p option so that mysql prompts for the password during the execution of the command.

Comment 1

3 years ago
It is indeed advisable to not enter password on the command line directly. 

Just
mysql_tzinfo_to_sql /usr/share/zoneinfo/ | mysql -uroot -p mysql
must prompt for a password and work.

Note that -uroot and -u root does the same thing.
Status: UNCONFIRMED → NEW
Ever confirmed: true

Comment 2

3 years ago
Commits pushed to master at https://github.com/mozilla/mozillians

https://github.com/mozilla/mozillians/commit/f009f12feed8cc4d8a4af90f39baef44714a4ce6
[fix bug 1234313] Remove password from the command line.

https://github.com/mozilla/mozillians/commit/1b7aebe9fbf047a6e4853ce05bf7c607bf69a2dd
Merge pull request #1277 from akatsoulas/1234313-update-doc

[fix bug 1234313] Remove password from the command line.

Updated

3 years ago
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED

Updated

3 years ago
Version: other → next
(Assignee)

Updated

3 years ago
Assignee: nobody → tasos
Verified as qa-. Although there is an issue with readthedocs builts in mozillians.org this should not affect it.
Status: RESOLVED → VERIFIED
Whiteboard: [profile-edit-refactor]
Version: next → 2016-1.1
You need to log in before you can comment on or make changes to this bug.