Bug 264813

Summary: Amarok: podcast episode download fails when temp filename is too long [patch]
Product: [Applications] amarok Reporter: Frank Steinmetzger <dev-kde>
Component: PodcastAssignee: Amarok Developers <amarok-bugs-dist>
Severity: normal CC: bart.cerneels
Priority: NOR    
Version: 2.4-GIT   
Target Milestone: 2.4.1   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In: 2.4.1
Attachments: Patch for shorter temporary filenames during download

Description Frank Steinmetzger 2011-01-30 01:57:27 UTC
Created attachment 56642 [details]
Patch for shorter temporary filenames during download

Version:           2.4-GIT (using KDE 4.6.0) 
OS:                Linux

Example podcast: http://www.heise.de/developer/podcast/itunes/heise-developer-podcast-softwarearchitektour.rss
This podcast has very long URIs to the episode files (example:  http://eu-heiseonline-div.s3.amazonaws.com/softwarearchitektour-26-systemsengineering.ane.bf41f3607069e50814dcfab7fdd391f0.mp3?x-sequenz=488;x-container=;x-filename=softwarearchitektour-26-systemsengineering.ane.bf41f3607069e50814dcfab7fdd391f0.mp3)

This results in a far too long temporary filename, hence Amarok can't download the episode. I'd like to provide this small patch with a quick fix. It uses a hash of the download URI as temporary filename.

Reproducible: Always
Comment 1 Myriam Schweingruber 2011-01-31 00:32:46 UTC
Could you please submit the patch to http://git.reviewboard.kde.org? You will need to get an identity on http://identity.kde.org before that.
Comment 2 Bart Cerneels 2011-02-01 10:19:48 UTC
Git commit 9ac8e9db34d7900f8ffc0a6e504b103ba04ae8ef by Bart Cerneels.
Committed on 01/02/11 at 10:15.
Pushed by shanachie into branch 'master'.

Podcast dowwnload temporary names could be to long.

Fixed by using MD5 hash as the name instead.
Patch by Frank Steinmetzger.


M  +2    -0    ChangeLog     
M  +7    -2    src/core-impl/podcasts/sql/SqlPodcastProvider.cpp