[wp-trac] [WordPress Trac] #57042: URL escaping added in 'about.php' file

WordPress Trac noreply at wordpress.org
Thu Nov 10 00:02:02 UTC 2022


#57042: URL escaping added in 'about.php' file
---------------------------+-----------------------------------------------
 Reporter:  monzuralam     |       Owner:  (none)
     Type:  defect (bug)   |      Status:  closed
 Priority:  normal         |   Milestone:
Component:  Help/About     |     Version:  6.1
 Severity:  normal         |  Resolution:  wontfix
 Keywords:  has-patch      |     Focuses:  administration, coding-standards
  close                    |
---------------------------+-----------------------------------------------
Changes (by peterwilsoncc):

 * status:  new => closed
 * resolution:   => wontfix
 * milestone:  6.1.1 =>


Comment:

 I agree this can be closed without a fix.

 While the intent is good, the escaping functions are only needed for user
 input. As the URL is hard coded in this instance, escaping isn't required.

 It's a nuanced point and I realise that the inconsistently in WordPress
 contributes to the confusion, so I really appreciate your taking the time
 to open the ticket @monzuralam

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/57042#comment:7>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list