[drupal-devel] [feature] Make default $db_url informative and unique

danielc drupal-devel at drupal.org
Wed Apr 13 20:10:34 UTC 2005


Issue status update for http://drupal.org/node/19800

 Project:      Drupal
 Version:      cvs
 Component:    base system
 Category:     feature requests
 Priority:     normal
 Assigned to:  Anonymous
 Reported by:  danielc
 Updated by:   danielc
 Status:       patch
 Attachment:   http://drupal.org/files/issues/dburl.per.install.diff (1.26 KB)

Ah.  Good catch.




danielc



Previous comments:
------------------------------------------------------------------------

April 1, 2005 - 00:21 : danielc

Attachment: http://drupal.org/files/issues/settings.dburl.diff (581 bytes)

Currently, the default $db_url in /drupal/sites/default/settings.php is
"mysql://drupal:drupal@localhost/drupal".  It would be helpful for end
users if its contents were more descriptive, such as
"mysql://user:password@localhost/databasename".


A side benefit of this change is helping automated installation scripts
determine if the settings in the /drupal/sites/default/ are meant to be
used or not.  This is because the existing db_url could possibly be
used in real life while the proposed text is very unlikely to be used.




------------------------------------------------------------------------

April 3, 2005 - 07:36 : Dries

This is clearly documented in the text above the setting.  Also, the
modifications are not consistent with the example in the documentation.




------------------------------------------------------------------------

April 3, 2005 - 13:44 : danielc

I'm concerned that some users could actually use the "drupal" for the
db, user and password.  That would trip up any bulid scripts that want
to ignore settings in sites/default/settings.php ONLY if they're not
really being used.  Plus using those as db/user/pw sucks for security.


How about using the sample URL from the docblock as the default $db_url
and I submit a patch to update the documentation with the new
information?




------------------------------------------------------------------------

April 5, 2005 - 12:23 : danielc

If the default $db_url does get changed, here is a list of Handbook
pages that make reference to $db_url:


http://drupal.org/node/260
http://drupal.org/node/17430
http://drupal.org/node/1357
http://drupal.org/node/495
http://drupal.org/node/5438
http://drupal.org/node/9407
http://drupal.org/node/3854


This last page makes mention of includes/conf.php instead of
sites/default/settings.php


Plus INSTALL.txt would need to be updated.




------------------------------------------------------------------------

April 7, 2005 - 13:50 : danielc

Attachment: http://drupal.org/files/issues/settings.dburl.per.install.diff (1.68 KB)

This patch unifies the format of the default $db_url.  It uses the
format found in INSTALL.txt, plus it clarifies the ability to change
the DMBS.




------------------------------------------------------------------------

April 11, 2005 - 17:46 : danielc

Can this please be applied to HEAD and 4-6?




------------------------------------------------------------------------

April 11, 2005 - 17:55 : Morbus Iff

-1. No user knows, or is told, what DBMS is.




------------------------------------------------------------------------

April 11, 2005 - 18:06 : danielc

If "dbms" is the concern, we can either explain what it means or can
leave the default as "mysql."  But, please, the default $db_url in
settings.php needs to be something else.  Tell me what you want it to
be and I'll make a patch with it.




------------------------------------------------------------------------

April 13, 2005 - 12:22 : danielc

This patch changes the default $db_url in settings.php to match the one
already in INSTALL.txt: mysql://userid:password@localhost/database




------------------------------------------------------------------------

April 13, 2005 - 12:24 : danielc

Attachment: http://drupal.org/files/issues/settings.dburl.per.install2.diff (822 bytes)

Uploading the patch helps, eh?




------------------------------------------------------------------------

April 13, 2005 - 12:26 : Morbus Iff

+1 from me.




------------------------------------------------------------------------

April 13, 2005 - 14:15 : Dries

Please update the phpdoc comments in settings.php as well.




------------------------------------------------------------------------

April 13, 2005 - 14:33 : danielc

The patch (settings.dburl.per.install2.diff, associated with comment 10)
does update the docblock.




------------------------------------------------------------------------

April 13, 2005 - 14:56 : Dries

It doens't update all occurences; search for 'db_name'.







More information about the drupal-devel mailing list