Ubuntu
rhythmbox package

Rhythmbox fails to download many podcasts

Bug #46799 reported by Roshan Shariff
42
Affects Status Importance Assigned to Milestone
Rhythmbox
Fix Released
Medium
rhythmbox (Ubuntu)
Fix Released
Medium
Ubuntu Desktop Bugs

Bug Description

Podcast episodes whose URLs contain a double-slash in the query string are not downloaded. The path is converted to a single slash in the HTTP request.

To reproduce, try subscribing to Leo Laporte's "This Week in Tech" at http://leo.am/podcasts/twit

The episodes whose URLs contain a double slash, like http://www.podtrac.com/pts/redirect.mp3?http://aolradio.podcast.aol.com/twit/TWiT0054H.mp3
fail to download. Right clicking on the episode and selecting the details tab shows the correct URL.

Ethereal shows that the following HTTP conversation takes place:

HEAD /pts/redirect.mp3?http:/twit.ord.cachefly.net/FLOSS-006.mp3 HTTP/1.1

Host: www.podtrac.com

User-Agent: gnome-vfs/2.14.1 neon/0.25.4

Connection: TE

TE: trailers

HTTP/1.1 404 Invalid Destination URL

Date: Fri, 26 May 2006 17:00:13 GMT

Server: Microsoft-IIS/6.0

X-Powered-By: ASP.NET

X-AspNet-Version: 1.1.4322

Transfer-Encoding: chunked

Cache-Control: private

Content-Type: text/html

0

Using Firefox to download that URL works fine, and produces the following exchange:

GET /pts/redirect.mp3?http://twit.ord.cachefly.net/FLOSS-006.mp3 HTTP/1.1

Host: www.podtrac.com

User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.8.0.3) Gecko/20060523 Ubuntu/dapper Firefox/1.5.0.3

Accept: text/xml,application/xml,application/xhtml+xml,text/html;q=0.9,text/plain;q=0.8,image/png,*/*;q=0.5

Accept-Language: en-us,en;q=0.5

Accept-Encoding: gzip,deflate

Accept-Charset: ISO-8859-1,utf-8;q=0.7,*;q=0.7

Keep-Alive: 300

Connection: keep-alive

HTTP/1.1 302 Found

Date: Fri, 26 May 2006 17:00:19 GMT

Server: Microsoft-IIS/6.0

X-Powered-By: ASP.NET

X-AspNet-Version: 1.1.4322

Location: http://twit.ord.cachefly.net/FLOSS-006.mp3

Cache-Control: private

Content-Type: text/html; charset=utf-8

Content-Length: 159

<html><head><title>Object moved</title></head><body>

<h2>Object moved to <a href='http://twit.ord.cachefly.net/FLOSS-006.mp3'>here</a>.</h2>

</body></html>

Note the conversion of the "//" to "/" in the query string of the first case.

I'm running Rhythmbox 0.9.3.1 on Ubuntu Dapper. I'm not sure if the problem is in Gnome-VFS or Rhythmbox

Simon Law (sfllaw)
Changed in rhythmbox:
status: Unconfirmed → Confirmed
Revision history for this message
FrancisMarkham (fmarkham-bugs) wrote :

upstream seem to think it is the 302 redirect that causes the problem.

Revision history for this message
Sebastien Bacher (seb128) wrote :

Thank you for commenting upstream about that

Changed in rhythmbox:
assignee: nobody → desktop-bugs
Bug Watch Updater (bug-watch-updater)
Changed in rhythmbox:
status: Confirmed → Fix Released
Revision history for this message
Daniel Holbach (dholbach) wrote :

Fixed upstream in CVS.

Changed in rhythmbox:
status: Confirmed → Fix Committed
Revision history for this message
Sebastien Bacher (seb128) wrote :

This upload fixes the issue:

" rhythmbox (0.9.6-0ubuntu1) edgy; urgency=low
 .
   * New upstream version, UVF exception granted by Matt Zimmerman:
     - fix DAAP seeking
     - display track numbers in playlists too
     - retrieve playlsit and audio folder information from HAL
     - make search box coloured when active
     - improve ID3 tag writing
     - load local cover art, and large improvements to plugin
     - build fixes: Solaris
     - Store MusicBrainz track IDs in the DB
     - fix some i18n issues
     - add gnome-keyring support to DAAP
     - allow hiding of plugins and disabler per-user plugins
     - plugin improvements
     - HTTP proxy-related fixes/improvements
     - drag-and-drop improvements
     - importing fixes
     - add "property changed" signal for dbus clients
     - turn CD burning into a plugin
     - turn audio CD support into a plugin
     - LIRC plugin fixes
     - improve database thread usage
     - make DB handle unknown entry-types
     - improve plugin debug output, and bindings
     - improve "import errors" and "missing files" source
     - fix many memory leaks
     - use GMapperFiles for DAAP transfers, fixing some iTunes clients
     - add support for alternate playlists files
     - handle transparent panels correctly
     - fix playlists subset searching
     - make iradio playlists handling better
     - use GOption instead og popt
     - remember visibility for remote files, as well as local ones
     - be smarter about creating podcast file names
     - allow multiple selections in the Import Folder dialog
     Plus the usual assortment of bug fixes, performance
     and memory improvements, and minor features.
   * debian/control.in:
     - Build-Depends on libgnome-keyring-dev for gnome-keyring DAAP use
     - updated pygtk requirement according to configure
   * debian/patches/02_autoconf.patch:
     - updated
   * debian/patches/03_dbus0.9.patch:
     - fixed with the new version
   * debian/patches/16_transparency_for_notication_area_icon.patch:
     - fixed with the new version
"

Changed in rhythmbox:
status: Fix Committed → Fix Released
Bug Watch Updater (bug-watch-updater)
Changed in rhythmbox:
importance: Unknown → Medium
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.