[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