[wp-trac] [WordPress Trac] #20759: Implement set_url_scheme() where appropriate

WordPress Trac wp-trac at lists.automattic.com
Fri Sep 14 18:37:59 UTC 2012


#20759: Implement set_url_scheme() where appropriate
-------------------------------------+-----------------------
 Reporter:  johnbillion              |       Owner:  ryan
     Type:  defect (bug)             |      Status:  reopened
 Priority:  normal                   |   Milestone:  3.5
Component:  General                  |     Version:  3.4
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch needs-testing  |
-------------------------------------+-----------------------
Changes (by mindctrl):

 * cc: mindctrl (added)
 * status:  closed => reopened
 * resolution:  fixed =>
 * type:  enhancement => defect (bug)


Comment:

 Something in this set of patches broke the Visit Site links in the admin
 bar when the site is SSL only. It strips out the https and instead of
 linking https://domain.com it only links domain.com (no http* - only
 relative). I've tested this on three separate installs and found this to
 be true.

 My test sites are all full-time SSL, with no option to access the sites
 via non-SSL means which results in the following error:



 400 Bad Request

 Your browser sent a request that this server could not understand.
 Reason: You're speaking plain HTTP to an SSL-enabled server port.
 Instead use the HTTPS scheme to access this URL, please.


 Shouldn't this not be stripping off the protocol at the front of the link?

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/20759#comment:15>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list