[wp-trac] [WordPress Trac] #13839: Configuration script does not allow apostrophes, blank prefixes
WordPress Trac
wp-trac at lists.automattic.com
Tue Aug 30 00:00:10 UTC 2011
#13839: Configuration script does not allow apostrophes, blank prefixes
----------------------------+-----------------------------
Reporter: incaren | Owner: dd32
Type: defect (bug) | Status: assigned
Priority: low | Milestone: Future Release
Component: Administration | Version: 2.9.2
Severity: trivial | Resolution:
Keywords: has-patch |
----------------------------+-----------------------------
Comment (by dd32):
> 13839.2.diff still doesn't allow apostrophes in a password, resulting
in "Error establishing a database connection".
I had to edit the patch to remove some other changes, so I must've removed
too much in the process.
I'm not really happy with the patch to start with, the changes to wp_die()
are half-baked in this patch, the refactoring in #18180 is really needed
IMO.
I'm going to rework the stripslashes()/addcslashes() changes specifically
from the patch and go from there I think, That'll at least fix this
tickets main issue. For consistancy, We should probably apply this to the
Hostname and the Prefix as well (as noted in #18180) even although
Apostrophes should never be in a hostname.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/13839#comment:9>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list