[bug] video-tag i.c.m. https

[afraca]

Omdat ik standaard de https variant van youtube gebruik (ja mensjes, die bestaat ) , maar niet zovaak video's embed op het forum, kwam ik er zojuist pas achter dat jullie regex voor de videotag geen https ondersteund.

Code om te reproduceren:

code:

[video]https://www.youtube.com/watch?v=11GYvfYjyV0[/video]

Als illustratie (bij bugfix commit zal dit dus wel een video opleveren):

[crisp]

Ik zie eigenlijk geen zwaarwegende redenen om dat te ondersteunen, alleen redenen om het juist niet te doen. Ik zou nog wel kunnen overwegen om https:// impliciet om te zetten naar http://

[AW_Bos]

..want op die manier krijg je toch weer die meldingen van 'mixed content' ?

[crisp]

AW_Bos schreef op zondag 17 april 2011 @ 19:36:
..want op die manier krijg je toch weer die meldingen van 'mixed content' ?

Dat speelt vziw alleen als je non-secured elementen mixed in een secured pagina, niet andersom

[Notna]

crisp schreef op zondag 17 april 2011 @ 20:56:
[...]

Dat speelt vziw alleen als je non-secured elementen mixed in een secured pagina, niet andersom

Yup, dat klopt

[AW_Bos]

Wat zijn dan de redenen om geen https toe te staan?
Dat het hele video-gebeuren via HTTPS geen echt nut heeft, snap ik wel .

[crisp]

AW_Bos schreef op zondag 17 april 2011 @ 21:55:
Wat zijn dan de redenen om geen https toe te staan?
Dat het hele video-gebeuren via HTTPS geen echt nut heeft, snap ik wel .

Juist omdat het niet nuttig is (zeker niet in dit specifieke geval omdat de http:// url natuurlijk net zo goed werkt), maar wel zorgt voor extra belasting zowel client- als serverside (in dit geval bij youtube).

[crisp]

Fix gemaakt: https voor youtube wordt ondersteund, maar omgezet naar http:

en alleen omdat ik RHCP ook cool vind