GNOME Bugzilla – Bug 155283
Rhythmbox doesn't encode URL properly making firefox misbehave
Last modified: 2005-09-04 03:38:43 UTC
(this is Debian http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=225088 ) - Configure firefox as your GNOME browser - Play a song in an Album or from an Artist with a space in its name - click on the allmusic link in the top of rhythmbox The problem is that firefox interprets "|" as a "Create new tab" separator, and allmusic.com only accepts "|" as a space between words. I tried to url_encode() the "|", but allmusic.com does not accept hex encoded chars in the URL (this is a real problem here, they're running Microsoft-IIS/5.0). I tried sending a %20, a %7C ("|"), and a + instead of the |, and the search worked in all cases, but the search title in the page contained the litteral characters I typed in the URL and this is UGLY! Now firefox is right in using | as an extension mechanism, and allmusic is wrong in not conforming to HTTP specs, what we can fix is rhythmbox, to send "+" instead of "|". It's ugly because "MOON+SAFARI" is displayed instead of "MOON SAFARI" in the search results, but it works. To do this, remove the two while loops with a '|' in widgets/rb-header.c, *_info_url will get the quoting right. Side note, it would be best to use a real URL encoding function.
*** Bug 160025 has been marked as a duplicate of this bug. ***
*** Bug 170929 has been marked as a duplicate of this bug. ***
*** Bug 304762 has been marked as a duplicate of this bug. ***
*** Bug 300976 has been marked as a duplicate of this bug. ***
I can't reproduce this with Rb 0.8.8 and Firefox 1.06 (even tried with Epiphany 1.6.4). Seems like Rb still uses | as a space, so maybe something in the way Firefox starts new tabs has changed?
We've switched to using last.fm, rather than allmusic.com, so this shouldn't happend any more.