[wp-trac] [WordPress Trac] #37454: get_avatar_data() delivers different url for scheme=https and is_ssl()

[WordPress Trac]

#37454: get_avatar_data() delivers different url for scheme=https and is_ssl()
----------------------------------------+---------------------
 Reporter:  neoxx                       |       Owner:  (none)
     Type:  defect (bug)                |      Status:  new
 Priority:  normal                      |   Milestone:  6.7
Component:  Users                       |     Version:  4.3
 Severity:  normal                      |  Resolution:
 Keywords:  has-patch needs-unit-tests  |     Focuses:
----------------------------------------+---------------------
Changes (by peterwilsoncc):

 * milestone:  Awaiting Review => 6.7


Comment:

 Reviving this as Gravatar now redirects all requests to HTTPS URLs. This
 can result in unnecessary 301 redirects.

 In the [https://github.com/WordPress/wordpress-develop/pull/6689 linked
 pull requests]:

 * retain the scheme setting for `get_avatar_data()` to account for other
 services
 * ignore the scheme setting when generating gravatar URLs
 * updates references in the tests to use secure.gravatar.com

 As the WordPress 6.6 beta is fast approaching, I've put this on the 6.7
 milestone. I think it would be fine to put in earlier but will allow the
 release squad to decide if they want to bring it forward.

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