[wp-trac] [WordPress Trac] #25449: wp_upload_dir() doesn't support https

WordPress Trac noreply at wordpress.org
Tue Mar 8 02:21:19 UTC 2016


#25449: wp_upload_dir() doesn't support https
---------------------------------------------+-----------------------------
 Reporter:  ryanhellyer                      |       Owner:
     Type:  defect (bug)                     |      Status:  new
 Priority:  normal                           |   Milestone:  Future Release
Component:  Upload                           |     Version:  3.8
 Severity:  major                            |  Resolution:
 Keywords:  needs-testing https needs-patch  |     Focuses:
---------------------------------------------+-----------------------------

Comment (by thomaswm):

 Replying to [comment:24 joemcgill]:
 > Reading back through the history of #15928 and #32112 makes me wonder if
 the best approach, as we prepare to be HTTPS first, is to remove the
 `is_admin()` checks that were introduced in r32342 and instead, add logic
 to `get_image_tag()` to always use HTTP URLs when the site url is setup
 using an HTTP scheme.

 See [attachment:25449.2.diff].

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


More information about the wp-trac mailing list