Make default $db_url informative and unique [#19800]

Comment	File	Size	Author
#15	dburl.per.install.diff	1.26 KB	danielc

#10	settings.dburl.per.install2.diff	822 bytes	danielc

#5	settings.dburl.per.install.diff	1.68 KB	danielc

	settings.dburl.diff	581 bytes	danielc

Comment #1

danielc CreditAttribution: danielc commented 1 April 2005 at 05:23

Title:

Change default $db_url to something unique and informative

» Make default $db_url informative and unique

Log in or register to post comments

Comment #2

Dries CreditAttribution: Dries commented 3 April 2005 at 12:36

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

Log in or register to post comments

Comment #3

danielc CreditAttribution: danielc commented 3 April 2005 at 18:44

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?

Log in or register to post comments

Comment #4

danielc CreditAttribution: danielc commented 5 April 2005 at 17:23

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.

Log in or register to post comments

Comment #5

danielc CreditAttribution: danielc commented 7 April 2005 at 18:50

File	Size
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.

Log in or register to post comments

Comment #6

danielc CreditAttribution: danielc commented 11 April 2005 at 22:46

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

Log in or register to post comments

Comment #7

Morbus Iff

they/them

CreditAttribution: Morbus Iff commented 11 April 2005 at 22:55

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

Log in or register to post comments

Comment #8

danielc CreditAttribution: danielc commented 11 April 2005 at 23:06

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.