Skip to:
Content

BuddyPress.org

Opened 8 years ago

Closed 8 years ago

#7220 closed defect (bug) (fixed)

Background image upload HTTPS SSL

Reported by: dono12's profile Dono12 Owned by: r-a-y's profile r-a-y
Milestone: 2.7 Priority: normal
Severity: normal Version: 2.4.0
Component: Media Keywords: dev-feedback has-patch
Cc:

Description

I recently added an SSL Certificate and

I’m getting a mixed content error on User Cover image uploads. When I inspect the image

the image link is missing the the ‘s’ from the http. This issue was addressed concerning
avatar uploads all is well there but cover image uploads are bonkers.

FOR AVATAR HTTPS ISSUE
https://buddypress.org/support/topic/buddypress-breaks-https/

AVATAR FIXED FOR SSL:

https://buddypress.trac.wordpress.org/changeset/5956

Can someone give me temporary solution,
in case it will be addressed in an Update. Background image upload?

Attachments (1)

7220.01.patch (608 bytes) - added by r-a-y 8 years ago.

Download all attachments as: .zip

Change History (4)

#1 @DJPaul
8 years ago

  • Component changed from Core to Media
  • Milestone changed from Awaiting Review to 2.7
  • Priority changed from omg pizza to normal
  • Severity changed from blocker to normal

Good find!

I don't know how possible it is to workaround at the moment. Maybe whoever works on the ticket can comment.

#2 @r-a-y
8 years ago

  • Keywords has-patch added; needs-patch removed
  • Version changed from 2.6.1.1 to 2.4.0

I've attached a patch that fixes this. Tested on a HTTPS site.

@dono12 - Give it a try and see if it fixes it for you as well.

@r-a-y
8 years ago

#3 @r-a-y
8 years ago

  • Owner set to r-a-y
  • Resolution set to fixed
  • Status changed from new to closed

In 11068:

Core: Use HTTPS when generating cover image URL.

Fixes #7220.

Note: See TracTickets for help on using tickets.