Our site is running in https.

The urls for the videos that are created are always http even if you enter in an https url like https://www.youtube.com/watch?v=pgTTMdxhmvw.

There needs to be a check if the url being inputed is https and if so output an https url. Or someway allow for https urls when the site is running in https

CommentFileSizeAuthor
#4 asset-fix-video-https-support.patch8.46 KBkiseleva.t
#3 asset-fix-video-https-support.patch8.46 KBkiseleva.t
#2 asset-fix-youtube-https-support-2054697-7940303.patch1.21 KBmattew
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mattew’s picture

mattew CreditAttribution: mattew commented
Priority: Normal » Major
Status: Active » Needs review

I created a patch for one of our project entirely working on https.
I think this issue is important.

mattew’s picture

mattew CreditAttribution: mattew commented
FileSize
asset-fix-youtube-https-support-2054697-7940303.patch1.21 KB
kiseleva.t’s picture

kiseleva.t CreditAttribution: kiseleva.t commented
Issue summary: View changes
FileSize
asset-fix-video-https-support.patch8.46 KB

By default youtube provide link to video without https:
Ex. http://youtu.be/pgTTMdxhmvw
If user insert it in asset, so video will be with http. I think need add check to secure page and replace http to https.
I add this check in module emvideo, so it will be replace http to https for all video providers, if we on secure page now.

kiseleva.t’s picture

kiseleva.t CreditAttribution: kiseleva.t commented
FileSize
asset-fix-video-https-support.patch8.46 KB

Sorry, previous file contains little formatting error.

IRuslan’s picture

IRuslan CreditAttribution: IRuslan commented
Status: Needs review » Closed (fixed)

Hi,

Patch from Tanya is good for all providers except INA. For INA provider need additional work, created issue for this #2318327: Ina provider not working anymore.

Patch asset-fix-video-https-support.patch committed to dev.