fossar / selfoss

multipurpose rss reader, live stream, mashup, aggregation web application
https://selfoss.aditu.de
GNU General Public License v3.0
2.38k stars 345 forks source link

No videos in RSS feeds #516

Open pieqq opened 10 years ago

pieqq commented 10 years ago

Tested on Selfoss 2.10.

When a feed article contains a video, it is not displayed.

Example: http://www.lense.fr/blog/feed/

It contains the following item:

<item>
<title>
DJI Phantom Vision 2+ : le nouveau drone qui survole la concurrence ?
</title>
<link>
http://www.lense.fr/2014/04/22/dji-phantom-vision-2-le-nouveau-drone-qui-survole-la-concurrence/
</link>
<comments>
http://www.lense.fr/2014/04/22/dji-phantom-vision-2-le-nouveau-drone-qui-survole-la-concurrence/#comments
</comments>
<pubDate>Tue, 22 Apr 2014 14:30:47 +0000</pubDate>
<dc:creator>ThierryR</dc:creator>
<category>
<![CDATA[ Matériel ]]>
</category>
<category>
<![CDATA[ DJI ]]>
</category>
<category>
<![CDATA[ drone ]]>
</category>
<category>
<![CDATA[ Phantom Vision 2+ ]]>
</category>
<category>
<![CDATA[ quadricoptere ]]>
</category>
<guid isPermaLink="false">http://www.lense.fr/?p=117369</guid>
<description>
<![CDATA[
Au delà des controverses, les drones sont en passe de révolutionner la prise de vue. La preuve avec le nouveau quadricoptère de DJI, toujours plus performant. &#8230;&#8230;&#8230;&#8230;&#8230;&#8230;&#8230;&#8230;&#8230;&#8230; Constructeur chouchou des vidéastes volants ces derniers mois, DJI vient de mettre la barre très haute avec le Phantom Vision 2+, qui repousse encore les possibilités tout en ...
]]>
</description>
<content:encoded>
<![CDATA[
<p>Au delà des controverses, les drones sont en passe de révolutionner la prise de vue. La preuve avec le nouveau <a href="http://www.dji.com/product/phantom-2-vision-plus">quadricoptère de DJI</a>, toujours plus performant.</p> <p><a href="http://www.lense.fr/2014/04/22/dji-phantom-vision-2-le-n
]]>
<![CDATA[
ouveau-drone-qui-survole-la-concurrence/dji-phantom-2-vision-quadcopter-703-p-650x397/" rel="attachment wp-att-117371"><img src="http://www.lense.fr/wp-content/uploads/2014/04/dji-phantom-2-vision-quadcopter-703-p-650x397-600x366.png" alt="" title="dji-phantom-2-vision-quadcopter-703-p-650x397" width="600" height="366" class="aligncenter size-medium wp-image-117371" /></a></p> <p><span id="more-117369"></span></p> <p style="text-align: center;">&#8230;&#8230;&#8230;&#8230;&#8230;&#8230;&#8230;&#8230;&#8230;&#8230;</p> <p>Constructeur chouchou des vidéastes volants ces derniers mois, DJI vient de mettre la barre très haute avec <a href="http://www.dji.com/product/phantom-2-vision-plus">le Phantom Vision 2+</a>, qui repousse encore les possibilités tout en mettant un sacré coup à la concurrence.<br /> Avec son stabilisateur d&#8217;images à trois axes, ce drone permet une prise de vue net, en douceur et sans mouvement parasite.<br /> Capable de résister à un vent de 40 km/h il promet des prise de vue aériennes avec une stabilité rare. Et si vous avez des doutes, voici la vidéo promo de la bête, qui risque de vous convaincre.</p> <p><iframe src="//player.vimeo.com/video/90986568" width="500" height="281" frameborder="0" webkitallowfullscreen mozallowfullscreen allowfullscreen></iframe> <p><a href="http://vimeo.com/90986568"></p> <p style="text-align: center;">&#8230;&#8230;&#8230;&#8230;&#8230;&#8230;&#8230;&#8230;&#8230;&#8230;</p> <p>D&#8217;un point de vue technique, le Phantom Vision 2+ embarque une caméra de 14 megapixel qui peut filmer en 1080p a 30 FPS ou en 720p à 60 FPS.<br /> Toute prise de vue se fait en temps réel, grâce à la caméra qui retransmet ce que le drone voit directement sur vos smartphones iOS ou Android.<br /> Et bien entendu, capable de se géolocaliser, ce drone reviendra au point GPS de son décollage lorsque la batterie viendra à faire défaut.</p> <p>Simple à piloter, capable de faire des prises de vues presque professionnelles et avec son prix abordable de 1299$, ce nouveau quadricoptere risque de faire trembler les rails de travelling, les grues ou les hélicoptères.<br /> La première compagnie qui proposera un tel drone avec le possibilité de mettre sa propre action camera ou son appareil photo gagnera la guerre des drones &#8230;</p> <p style="text-align: center;">&#8230;&#8230;&#8230;&#8230;&#8230;&#8230;&#8230;&#8230;&#8230;&#8230;</p> <p>+ <a href="http://www.dji.com/product/phantom-2-vision-plus">DJI </a></p> <p>+ via <a href="http://www.diyphotography.net/dji-announces-phantom-vision-2-latest-addition-drone-line/">DIYphotography</a> </p>
]]>
</content:encoded>
<wfw:commentRss>
http://www.lense.fr/2014/04/22/dji-phantom-vision-2-le-nouveau-drone-qui-survole-la-concurrence/feed/
</wfw:commentRss>
<slash:comments>1</slash:comments>
</item>

As you can see, inside the content there is an iframe and a link to the video (on Vimeo)

<p><iframe src="//player.vimeo.com/video/90986568" width="500" height="281" frameborder="0" webkitallowfullscreen mozallowfullscreen allowfullscreen></iframe> <p><a href="http://vimeo.com/90986568"></p>

However, when I display the article, nothing is shown (I put a red eclipse where the video is supposed to be displayed):

No video shown

Is there a way to display the videos in the RSS feeds?

niol commented 10 years ago

Maybe add the iframe element in helpers/ContentLoader.php in function sanitizeContent().

SSilence commented 10 years ago

I think thats not a good idea, because of security reasons.

pieqq commented 10 years ago

Thank you guys for your comments. You're right @SSilence, after @niol explained how to show it, I understood it was removed for a reason...

Do you think it would be possible to show a placeholder where an element like a video was supposed to show? So I know I have to view the real article to see the video... Or even better, in the case of iframes from youtube/dailymotion/vimeo, to extract the URL of the video and show it, so the user just has to click on it to view the video.

What do you think?

For Youtube, an embed link looks like this:

<iframe width="420" height="315" src="//www.youtube.com/embed/clC6cgoh1sU" frameborder="0" allowfullscreen></iframe>

For Vimeo:

<iframe src="//player.vimeo.com/video/90312869" width="500" height="281" frameborder="0" webkitallowfullscreen mozallowfullscreen allowfullscreen></iframe> <p><a href="http://vimeo.com/90312869">360° Video using 6 GoPro Cameras - spherical panorama timelapse</a> from <a href="http://vimeo.com/j0n4s">j0n4s</a> on <a href="https://vimeo.com">Vimeo</a>.</p>

For Dailymotion:

<iframe frameborder="0" width="480" height="270" src="//www.dailymotion.com/embed/video/xqdlrf" allowfullscreen></iframe><br /><a href="http://www.dailymotion.com/video/xqdlrf_sintel_shortfilms" target="_blank">Sintel</a> <i>发布人 <a href="http://www.dailymotion.com/flattr" target="_blank">flattr</a></i>

Basically, all of them use an iframe with a src element. Maybe it's possible to filter and display iframes only coming from youtube.com, vimeo.com and dailymotion.com for instance?

EHLOVader commented 10 years ago

I would like to see this as well. In fact I would like full MRSS support.

I understand the potential for identifying location if you load videos directly. And also understand the security concerns surrounding the use of iframes.

But I think that those should be concerns left to the end user and an opt in option to allow videos or some content to be shown or parsed for rendering in the best way possible for the end users.

SSilence commented 10 years ago

For security reasons this should be a whitelist, e.g. allow iframes with embed link of youtube, vimeo, ...

pieqq commented 10 years ago

Good news! Thanks Tobias! Do you think this whitelist should be in the config file or in the Settings page?