Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lien fichier média #504

Closed
Cypouz opened this issue May 13, 2014 · 15 comments
Closed

Lien fichier média #504

Cypouz opened this issue May 13, 2014 · 15 comments
Assignees
Milestone

Comments

@Cypouz
Copy link

Cypouz commented May 13, 2014

Lorsqu'un flux RSS contient le lien d'un fichier média — typiquement un MP3 dans le cas d'un podcast — il serait bien de conserver l'affichage de ce lien dans FreshRSS afin de ne pas avoir à aller sur le site d'origine pour le télécharger.

Dans l'idéal, un lecteur audio HTML5 intégré pourrait permettre de l'écouter directement depuis FreshRSS.

@aledeg
Copy link
Member

aledeg commented May 13, 2014

Pourrais tu donner un exemple pour les tests? Merci

@Cypouz
Copy link
Author

Cypouz commented May 13, 2014

Bien-sûr. À tout hasard : www.captainposix.net/RSSPodcast/feeditunes.xml

@aledeg
Copy link
Member

aledeg commented May 13, 2014

merci

Alkarex added a commit that referenced this issue May 13, 2014
@Alkarex Alkarex added this to the 0.8.0 milestone May 13, 2014
@Alkarex Alkarex self-assigned this May 13, 2014
@Alkarex
Copy link
Member

Alkarex commented May 13, 2014

Voilà qui est fait 7f1c305
podcast
Il reste en particulier un peu de style, voir si on fait pareil pour les vidéos (exemples de flux avec vidéos en enclosure ?), et mettre à jour le journal des changements.

@Cypouz
Copy link
Author

Cypouz commented May 13, 2014

T'ain vous êtes trop forts les gars \o/ J'en ai la larme à l'œil :'-)

Voici le flux du 56Kast, qui mixe à la fois des fichiers audio et vidéo : www.liberation.fr/podcast/40

@Alkarex
Copy link
Member

Alkarex commented May 13, 2014

Arf, les enclosure vidéo de ce flux (56Kast) sont invalides : elles indiquent "video/mp4" alors qu'il s'agit d'un lien vers une page HTML (chez DailyMotion)...

@Cypouz
Copy link
Author

Cypouz commented May 13, 2014

Effectivement. Celui-là, alors : http://geekbrief.podshow.com/feed.xml

Alkarex added a commit that referenced this issue May 14, 2014
@Alkarex
Copy link
Member

Alkarex commented May 14, 2014

Et voilà c3178a4
video

Alkarex added a commit that referenced this issue May 17, 2014
@Alkarex
Copy link
Member

Alkarex commented May 17, 2014

Bon, cela semble fonctionner

@Alkarex Alkarex closed this as completed May 17, 2014
@Cypouz
Copy link
Author

Cypouz commented Oct 2, 2014

Pour le 56Kast, les fichiers MP3 n'apparaissent pas alors qu'ils sont bien dans le flux : http://www.liberation.fr/podcast/40/

@Cypouz
Copy link
Author

Cypouz commented Oct 3, 2014

Idem pour Place de la Toile : http://radiofrance-podcast.net/podcast09/rss_10465.xml

@marienfressinaud
Copy link
Member

J'ai bien le problème pour le 56Kast mais pas pour Place de la Toile… Pour le 56Kast je suppose à première vue que c'est que les balises sont mal créées et que SimplePie ne les détecte pas comme il faut mais je n'en suis pas sûr…

@Alkarex Alkarex added the Work in progress 🚧 Wait before merging label Oct 5, 2014
@Alkarex
Copy link
Member

Alkarex commented Oct 5, 2014

J'y travaille. C'est un bug SimplePie

Alkarex added a commit that referenced this issue Oct 5, 2014
@Alkarex Alkarex removed the Work in progress 🚧 Wait before merging label Oct 5, 2014
@Alkarex
Copy link
Member

Alkarex commented Oct 5, 2014

Voilà qui est corrigé 70131f7
C'était un problème de double déclaration dans le flux RSS, avec une des deux déclarations qui n'indiquait pas le type MIME.
Les MP3 de http://www.liberation.fr/podcast/40/ devraient fonctionner maintenant.
Les vidéos qui ne fonctionnent pas, c'est parce qu'elles sont indiquées en video/mp4 alors que ce sont des pages HTML.

@Cypouz
Copy link
Author

Cypouz commented Oct 5, 2014

Cool, merci :)

Alkarex added a commit that referenced this issue Oct 9, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants