[wp-trac] [WordPress Trac] #46673: Update esc_url function (default https).

WordPress Trac noreply at wordpress.org
Mon Dec 6 15:25:28 UTC 2021


#46673: Update esc_url function (default https).
--------------------------+-----------------------
 Reporter:  aksl95        |       Owner:  (none)
     Type:  defect (bug)  |      Status:  reopened
 Priority:  normal        |   Milestone:
Component:  Formatting    |     Version:
 Severity:  normal        |  Resolution:
 Keywords:                |     Focuses:
--------------------------+-----------------------
Changes (by pcarvalho):

 * status:  closed => reopened
 * resolution:  maybelater =>


Comment:

 Can this be reviewed?

 Issues:
 1- Having the http:// hardcoded makes less sense here, it doesn't look up
 to $protocols.

 2- There's no way to use this function if I want to have https:// as
 default instead.

 3- I'm also not sure about
 " any website available over HTTPS is also available over HTTP and likely
 will be for the foreseeable future."

 I would suggest then that it pops the first allowed $protocols.

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


More information about the wp-trac mailing list