Allow embedding of external videos and Google things over HTTPS

Bug #809058 reported by François Marier
18
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
High
Eugene

Bug Description

Our removal of httpswwwroot in bug #646713 means that any external content must be available over https. Blocks that need fixing:

- external video
- google apps

Greg Hanley says:

"I just recently moved our site from always https to only login, because when students embed video from outside of the site (which they are encouraged to do to save file space and to take advantage of other resources) Internet Explorer will block the content every time you reload the page. It also looks like it is blocking the new Google Apps block type, even if the provided url/embed-code has https in it. As Iñaki Arenaza mentioned, we use LDAP for logins that can't be allowed to pass in clear text so HTTPS is important. At least 25% of our users use IE and it becomes very irritating having to constantly confirm "show all content" when editing a page or browsing a collection.

Some of this might be mitigated if the googleapps and externalvideo block types took into consideration the site's SSL status and embedded the content with https sources when available (both YouTube and Google Apps seem to support embedding over https)."

Tags: https
Changed in mahara:
milestone: none → 1.4.1
importance: Undecided → High
status: New → Triaged
Eugene (eugenev)
Changed in mahara:
assignee: nobody → Eugene (eugenev)
Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/564
Committed: http://gitorious.org/mahara/mahara/commit/9a2cfb0680c721841493cbe0f88a57c6dd2ed37a
Submitter: Francois Marier (<email address hidden>)
Branch: master

commit 9a2cfb0680c721841493cbe0f88a57c6dd2ed37a
Author: Eugene Venter <email address hidden>
Date: Mon Aug 8 11:32:58 2011 +1200

    Embed external videos and Google Apps over https

    Bug #809058

    If the site is running https, embed external videos and google apps
    over https where possible.

    Change-Id: Id1d4b66c1f04587d4864126c2e58d2167e45fe70
    Signed-off-by: Eugene Venter <email address hidden>

Changed in mahara:
status: Triaged → In Progress
milestone: 1.4.1 → 1.5.0
status: In Progress → Fix Committed
Vibhav Pant (vibhavp)
no longer affects: mahara (Ubuntu)
Melissa Draper (melissa)
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.