Here's a patch to set it back.

CommentFileSizeAuthor
settings.php__0.patch638 byteswebchick
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

webchick’s picture

Bump. This is still an issue and it's still critical; prevents you from installing Drupal HEAD at all unless you manually change it back to username:password, etc.

Steven’s picture

Status: Reviewed & tested by the community » Fixed

Appears to be fixed in HEAD?

webernet’s picture

Status: Fixed » Reviewed & tested by the community

Still not fixed on HEAD (6.x).

Steven’s picture

Status: Reviewed & tested by the community » Fixed
fantabulous:~/Projects/Drupal/drupal steven$ cvs up -C .
fantabulous:~/Projects/Drupal/drupal steven$ cat sites/default/settings.php | grep 'db_url'
 * Note that the $db_url variable gets parsed using PHP's built-in
 * $db_url parts, you can escape them via URI hex encodings:
 * of $db_url variables with the 'default' element used until otherwise
 *   $db_url = 'mysql://username:password@localhost/databasename';
 *   $db_url = 'mysqli://username:password@localhost/databasename';
 *   $db_url = 'pgsql://username:password@localhost/databasename';
$db_url = 'mysql://username:password@localhost/databasename';

What?

Steven’s picture

Gah, for some reason parts of my CVS tree had become sticky? I hate CVS.

Anonymous’s picture

Status: Fixed » Closed (fixed)
webernet’s picture

Title: settings.php change sneaked in » settings.php username:password changed
Status: Closed (fixed) » Reviewed & tested by the community

Looks like it's happened again...

ChrisKennedy’s picture

ChrisKennedy’s picture

Status: Reviewed & tested by the community » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)