On Tue, Nov 30, 2010 at 9:49 PM, John Goerzen <jgoer...@complete.org> wrote: > > Hi Avi, > > Thank you for the report and patch. I am not certain, however, that it's the > right thing to do. The feed in question contains: > > <enclosure > url="http://www.tor.com//images/stories/stories/Gilman/Lightbringers_and_Rainmakers/Gilman, > Felix - Lightbringers and Rainmakers.mp3" length="49306025" > type="audio/mpeg"/> > > in other words, the URL as given is incorrect (it should be using %20 instead > of space right there in the feed).
I see. So the comma has nothing to do with it, and the problem is with the original feed. > > won't your patch break compliant feeds by, say, replacing %20 by %2520? You're right. It will. I just verified that. So I guess my bug report should be re-tagged as a wishlist item: "Please allow download of un-escaped url's". But I see how it would be troublesome to implement (maybe avoid escaping if the url contains % characters ?), and, unfortunately, I lack the haskell mojo to suggest a patch. Thanks, Avi > > - John -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org