? Success

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
1 Sep 2016

Pull Request for Issue #11878

Summary of Changes

There is no need to default to root as no public webserver should allow access via root and empty pw.
Also no other cms database class defaults to root if it is not set. they just default to empty.

Testing Instructions

Documentation Changes Required

None.

avatar zero-24 zero-24 - open - 1 Sep 2016
avatar zero-24 zero-24 - change - 1 Sep 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 1 Sep 2016
Category Libraries
avatar zero-24 zero-24 - change - 1 Sep 2016
Title
remove the defaulting to the root account
[installation] Remove the defaulting to the root account
Labels Added: ?
avatar zero-24 zero-24 - edited - 1 Sep 2016
avatar zero-24 zero-24 - edited - 1 Sep 2016
avatar zero-24 zero-24 - edited - 1 Sep 2016
avatar zero-24 zero-24 - change - 1 Sep 2016
Title
[installation] Remove the defaulting to the root account
[installation] Remove the defaulting to the root account in mysql & mysqli
avatar rdeutz
rdeutz - comment - 2 Sep 2016

Maybe we remove localhost also?

avatar alikon
alikon - comment - 2 Sep 2016

tested on

  • PHP Built On Windows NT ALIKON-PC 6.1 build 7601 (Windows 7 Professional Edition Service Pack 1) i586
  • Database Version 5.5.5-10.1.8-MariaDB-log
  • Database Collation utf8_general_ci
  • Database Connection Collation utf8mb4_general_ci
  • PHP Version 5.5.30
  • Web Server Apache/2.4.17 (Win32) OpenSSL/1.0.2d PHP/5.5.30

and still able to install with user:root and an empty psw (as is quite common default config for tools like XAMPP)

Maybe we remove localhost also?

localhost

in this case maybe we need to change too

This is usually "localhost".
to
Either something as "localhost" or a hostname given by the host.

or something similar i guess

avatar alikon alikon - test_item - 2 Sep 2016 - Tested successfully
avatar alikon
alikon - comment - 2 Sep 2016

I have tested this item successfully on 775974e


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11881.

avatar sonalitailored sonalitailored - test_item - 2 Sep 2016 - Tested successfully
avatar sonalitailored
sonalitailored - comment - 2 Sep 2016

I have tested this item successfully on 775974e


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11881.

avatar sonalitailored sonalitailored - test_item - 2 Sep 2016 - Tested successfully
avatar sonalitailored
sonalitailored - comment - 2 Sep 2016

I have tested this item successfully on 775974e

I have tested this item successfully


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11881.

avatar brianteeman brianteeman - test_item - 2 Sep 2016 - Tested successfully
avatar brianteeman
brianteeman - comment - 2 Sep 2016

I have tested this item successfully on 775974e


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11881.

avatar brianteeman brianteeman - change - 2 Sep 2016
Status Pending Ready to Commit
avatar brianteeman
brianteeman - comment - 2 Sep 2016

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11881.

avatar joomla-cms-bot joomla-cms-bot - change - 2 Sep 2016
Labels Added: ?
avatar truptikagathara truptikagathara - test_item - 2 Sep 2016 - Tested successfully
avatar truptikagathara
truptikagathara - comment - 2 Sep 2016

I have tested this item successfully on 775974e


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11881.

avatar jeckodevelopment jeckodevelopment - test_item - 2 Sep 2016 - Tested successfully
avatar jeckodevelopment
jeckodevelopment - comment - 2 Sep 2016

I have tested this item successfully on 775974e


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11881.

avatar wilsonge wilsonge - change - 2 Sep 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-09-02 11:17:31
Closed_By wilsonge
avatar wilsonge wilsonge - close - 2 Sep 2016
avatar wilsonge wilsonge - merge - 2 Sep 2016
avatar joomla-cms-bot joomla-cms-bot - close - 2 Sep 2016
avatar joomla-cms-bot joomla-cms-bot - change - 2 Sep 2016
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment