After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 791154 - rss feed only downloads image, no audio
rss feed only downloads image, no audio
Status: RESOLVED FIXED
Product: totem-pl-parser
Classification: Core
Component: General
3.26.x
Other Linux
: Normal normal
: ---
Assigned To: totem-pl-parser-maint
totem-pl-parser-maint
: 794333 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2017-12-03 06:56 UTC by nat
Modified: 2018-05-04 18:31 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
rss parsing problem -- png instead of mp3 file (41.15 KB, image/png)
2017-12-13 06:41 UTC, nat
Details

Description nat 2017-12-03 06:56:10 UTC
all KQED feeds -- which have an image within the feed can't download audio. They only get the image. The rss use to work before Ubuntu update.

http://www.kqed.org/community/rss/
https://ww2.kqed.org/news/tag/tcrarchive/feed/podcast
https://ww2.kqed.org/perspectives/feed/podcast/

Also, their feed work using other software.

I contacted them, but they are suggesting to use other software.
Comment 1 Jonathan Matthew 2017-12-03 07:18:29 UTC
totem-pl-parser is responsible for extracting urls from rss feeds, so I'm moving this bug there.

possibly checking the 'medium' attribute on the media:content tag would help?

<media:content url="https://ww2.kqed.org/perspectives/wp-content/uploads/sites/44/2017/02/Peter-Gavin-New-800x800.jpg" medium="image" height="1920" width="1920">
Comment 2 nat 2017-12-03 16:04:12 UTC
Here is a sample of a media content tag:

<item>
		<title>The California Report</title>
		<link>https://ww2.kqed.org/news/2017/11/30/the-california-report-1804/</link>
		<pubDate>Thu, 30 Nov 2017 16:05:45 +0000</pubDate>
		<guid isPermaLink="false">https://ww2.kqed.org/news/?p=11634023</guid>
		<comments>https://ww2.kqed.org/news/2017/11/30/the-california-report-1804/#respond</comments>
		<wfw:commentRss>https://ww2.kqed.org/news/2017/11/30/the-california-report-1804/feed/</wfw:commentRss>
		<slash:comments>0</slash:comments>
		<description>Stanford Sexual Assault Case Re-enters Spotlight Stanford University says it will let stand a memorial resolution honoring a professor who was disciplined for sexual misconduct. The California Report talked with a woman about her experience when she was a graduate student. Reporter: Vanessa Rancano L.A. City Council Votes to Strengthen City’s Sexual Harassment Policies Council … &lt;a href=&quot;https://ww2.kqed.org/news/2017/11/30/the-california-report-1804/&quot; class=&quot;more-link&quot;&gt;Continue reading &lt;span class=&quot;screen-reader-text&quot;&gt;The California Report&lt;/span&gt; &lt;span class=&quot;meta-nav&quot;&gt;→&lt;/span&gt;&lt;/a&gt;</description>
		<enclosure url="https://www.podtrac.com/pts/redirect.mp3/www.kqed.org/.stream/anon/radio/tcr/2017/11/TCRAM20171130Podcast.mp3" length="6648496" type="audio/mpeg" />
		<itunes:subtitle>Stanford Sexual Assault Case Re-enters Spotlight Stanford University says it will let stand a memorial resolution honoring a professor who was disciplined for sexual misconduct. The California Report talked with a woman about her experience when she wa...</itunes:subtitle>
		<itunes:summary>Stanford Sexual Assault Case Re-enters Spotlight Stanford University says it will let stand a memorial resolution honoring a professor who was disciplined for sexual misconduct. The California Report talked with a woman about her experience when she was a graduate student. Reporter: Vanessa Rancano L.A. City Council Votes to Strengthen City’s Sexual Harassment Policies Council … Continue reading The California Report →</itunes:summary>
		<itunes:author>KQED</itunes:author>
		<itunes:image href="https://ww2.kqed.org/news/wp-content/uploads/sites/10/2017/07/The_California_Report_iTunes_Badge-06.png" />
		<itunes:duration>13:51</itunes:duration>
<post-id xmlns="com-wordpress:feed-additions:1">11634023</post-id><media:content url="https://ww2.kqed.org/news/wp-content/uploads/sites/10/2017/07/tcr_archive_background-800x300.jpg" medium="image" height="450" width="1200"><media:thumbnail url="https://ww2.kqed.org/news/wp-content/uploads/sites/10/2017/07/tcr_archive_background-160x60.jpg" height="60" width="60" /></media:content>
<nprml:parent id="319418027" type="collection"/>Stanford Sexual Assault Case Re-enters Spotlight Stanford University says it will let stand a memorial resolution honoring a professor who was disciplined for sexual misconduct. The California Report talked with a woman about her experience when she was a graduate student. Reporter: Vanessa Rancano L.A. City Council Votes to Strengthen City&#8217;s Sexual Harassment Policies Council &#8230; <a href="https://ww2.kqed.org/news/2017/11/30/the-california-report-1804/" class="more-link">Continue reading <span class="screen-reader-text">The California Report</span> <span class="meta-nav">&#8594;</span></a>
</item>
Comment 3 Bastien Nocera 2017-12-04 14:59:01 UTC
commit 686b64f9eba5e80ef8d53ca3041ae48c88b96b5a
Author: Bastien Nocera <hadess@hadess.net>
Date:   Mon Dec 4 15:55:39 2017 +0100

    podcast: Add support for "medium" property
    
    The medium property, similarly to the "type" property allows to define
    the type of additional resources, such as whether they are images or
    audio.
    
    This caused problems with KQED podcast feeds as we did not handle that
    property, and tried to use the image as the audio source.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=791154


Before:
** Message: Started playlist "file:///home/hadess/Projects/jhbuild/totem-pl-parser/plparse/tests/791154-kqed.rss"...
** Message: 	title = 'KQED’s The California Report'
** Message: 	language = 'en-US'
** Message: 	description = 'KQED Public Media for Northern CA'
** Message: 	author = 'KQED'
** Message: 	copyright = 'Copyright © 2017 KQED Inc. All Rights Reserved.'
** Message: 	publication-date = '2017-12-04T08:01:09Z' (1512374469/'Mon, 04 Dec 2017 08:01:09 +0000')
** Message: Added URI "https://ww2.kqed.org/news/wp-content/uploads/sites/10/2017/11/TCRMag_iTunes_3000x3000-800x800.jpg"...
** Message: 	author = 'KQED'
** Message: 	id = 'https://ww2.kqed.org/news/?p=11634383'
** Message: 	filesize = '13906776'
** Message: 	title = 'The California Report Magazine: Homeless Housemates and Gold Rush Opera'
** Message: 	publication-date = '2017-12-02T01:59:12Z' (1512179952/'Sat, 02 Dec 2017 01:59:12 +0000')
** Message: 	duration = '28:58'
** Message: 	description = 'Helping the Homeless, By Living With them, A Reporter and A Politician Share Loss, Giant Tree Stump Takes the Opera Stage, Bumpa' (truncated)
** Message: Playlist "file:///home/hadess/Projects/jhbuild/totem-pl-parser/plparse/tests/791154-kqed.rss" ended.

After:
** Message: Started playlist "file:///home/hadess/Projects/jhbuild/totem-pl-parser/plparse/tests/791154-kqed.rss"...
** Message: 	title = 'KQED’s The California Report'
** Message: 	language = 'en-US'
** Message: 	description = 'KQED Public Media for Northern CA'
** Message: 	author = 'KQED'
** Message: 	copyright = 'Copyright © 2017 KQED Inc. All Rights Reserved.'
** Message: 	publication-date = '2017-12-04T08:01:09Z' (1512374469/'Mon, 04 Dec 2017 08:01:09 +0000')
** Message: Added URI "https://www.podtrac.com/pts/redirect.mp3/www.kqed.org/.stream/anon/radio/tcrmag/2017/12/TCRPodcastDec1.mp3"...
** Message: 	image-url = 'https://ww2.kqed.org/news/wp-content/uploads/sites/10/2017/11/TCRMag_iTunes_3000x3000-800x800.jpg'
** Message: 	author = 'KQED'
** Message: 	id = 'https://ww2.kqed.org/news/?p=11634383'
** Message: 	filesize = '13906776'
** Message: 	title = 'The California Report Magazine: Homeless Housemates and Gold Rush Opera'
** Message: 	publication-date = '2017-12-02T01:59:12Z' (1512179952/'Sat, 02 Dec 2017 01:59:12 +0000')
** Message: 	duration = '28:58'
** Message: 	description = 'Helping the Homeless, By Living With them, A Reporter and A Politician Share Loss, Giant Tree Stump Takes the Opera Stage, Bumpa' (truncated)
** Message: Playlist "file:///home/hadess/Projects/jhbuild/totem-pl-parser/plparse/tests/791154-kqed.rss" ended.
Comment 4 nat 2017-12-13 06:29:14 UTC
I am running Rhythmbox 3.4.1-2ubuntu5. It is still not downloading the feeds. Was there an update to the code for me to check? Sorry - had to drop the problem, life in Ojai and had to evacuate from the fire.
Comment 5 nat 2017-12-13 06:41:15 UTC
Created attachment 365474 [details]
rss parsing problem -- png instead of mp3 file

shows how somehow the Rhythmbox gets the png image file instead of the mp3 audio file.
Comment 6 Bastien Nocera 2017-12-13 11:51:04 UTC
You'll need to ask your distribution to include this fix in their version.
Comment 7 nat 2017-12-13 16:21:28 UTC
How would I go about "asking my distribution to include this fix"? I am running Ubuntu 17.10. Thank you.
Comment 8 Sebastien Bacher 2017-12-13 16:42:03 UTC
@nat, open a bug on launchpad (https://bugs.launchpad.net/ubuntu/+source/totem-pl-parser) describing the issue and requesting the commit to be backported
Comment 9 nat 2017-12-14 00:46:23 UTC
I reported the bug on launchpad.net -- this is the link: https://bugs.launchpad.net/ubuntu/+source/totem-pl-parser/+bug/1738110
Comment 10 Jonathan Matthew 2018-03-26 21:40:10 UTC
*** Bug 794333 has been marked as a duplicate of this bug. ***
Comment 11 Jonathan Ryshpan 2018-05-04 17:18:10 UTC
Fedora is still not up to date.  Where are latest source and RPMs?  (Embarrassed to show my ignorance.)
Comment 12 Bastien Nocera 2018-05-04 17:41:41 UTC
(In reply to Jonathan Ryshpan from comment #11)
> Fedora is still not up to date.  Where are latest source and RPMs? 
> (Embarrassed to show my ignorance.)

There's no release with this bug fixed, yet.
Comment 13 Jonathan Ryshpan 2018-05-04 18:31:13 UTC
(In reply to Bastien Nocera from comment #12)
> (In reply to Jonathan Ryshpan from comment #11)
> > Fedora is still not up to date.  Where are latest source and RPMs? 
> > (Embarrassed to show my ignorance.)
> 
> There's no release with this bug fixed, yet.

Where is the source?  Is it practical for people other than Gnome developers to compile totem-pl-parser, or will compiling it require recompiling most of Gnome?

Thanks for the quick reply.