totem-mozilla doesn't work when the URL has whitespace chars

Bug #313686 reported by Ricardo Pérez López
36
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Totem
Fix Released
Medium
libsoup2.4 (Ubuntu)
Fix Released
Low
Ubuntu Desktop Bugs
Jaunty
Fix Released
Low
Ubuntu Desktop Bugs

Bug Description

Binary package hint: totem-mozilla

When the URL has whitespace chars, the totem-mozilla plugin doesn't work.

For example, try going to the following web page:

http://www.gameprotv.com/the-conduit-video-5532.html

The URL of the embedded video is:

http://videos.gameprotv.com/videos/The_Conduit/The Conduit Avance 592x336.wmv

(notice the blanks). The video can't be played. However, try the following:

http://www.gameprotv.com/mando-de-nintendo-revolution-video-1413.html

The URL of the embedded video is:

http://videos.gameprotv.com/videos/Mando_de_Nintendo_Revolution/GPTV_Mando_Nintendo_Revolution.wmv

(without blanks). This time, the video is played perfectly.

TESTCASE: verify that the video plays after upgrade

Revision history for this message
goto (gotolaunchpad) wrote :

I can confirm this.
Not sure though if whitespaces are actually legal. Like " and so on.
I think there is a rfc or so that explains that a whitespace should be %20 iirc.
If I try to open
http://videos.gameprotv.com/videos/The_Conduit/The%20Conduit%20Avance%20592x336.wmv
it works. So maybe this is a firefox or firefox plugin issue, because it does not convert whitespaces into %20.
Thanks!

Changed in totem:
status: New → Confirmed
Revision history for this message
Ricardo Pérez López (ricardo) wrote :

Confirmed by another user. Submitted to upstream.

Changed in totem:
assignee: nobody → desktop-bugs
importance: Undecided → Low
status: Confirmed → Triaged
Changed in totem:
status: Unknown → New
Revision history for this message
Sebastien Bacher (seb128) wrote :

the bug has been fixed to svn and the new version will be in jaunty

affects: totem (Ubuntu) → libsoup2.4 (Ubuntu)
Changed in libsoup2.4 (Ubuntu):
status: Triaged → Fix Committed
Revision history for this message
Ricardo Pérez López (ricardo) wrote :

Sebastien, the bug is still reproducible on my Jaunty, using the two web pages I posted in the bugreport description.

The following still doesn't work:

http://www.gameprotv.com/the-conduit-video-5532.html

The following works OK, as before:

http://www.gameprotv.com/mando-de-nintendo-revolution-video-1413.html

May I need to wait for a package update?

Changed in totem:
status: New → Fix Released
Revision history for this message
Sebastien Bacher (seb128) wrote :
description: updated
Revision history for this message
Martin Pitt (pitti) wrote :

Accepted libsoup2.4 into jaunty-proposed-proposed; please test and give feedback here. Please see https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

tags: added: verification-needed
Revision history for this message
Pedro Villavicencio (pedro) wrote :

The video indeed didn't worked with libsoup shipped on Jaunty, after update to the one in the proposed repositories i can see the video without any issues, the bug is fixed, thanks you.

tags: added: verification-done
removed: verification-needed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package libsoup2.4 - 2.26.0-0ubuntu3

---------------
libsoup2.4 (2.26.0-0ubuntu3) jaunty-proposed; urgency=low

  * debian/patches/90_svn_change_fix_locking_issue.patch:
    - svn change to fix a locking issue leading to some evolution crashers
      (lp: #340785)
  * debian/patches/91_svn_change_fix_unencoded_spaces_use.patch:
    - svn change to fix the loading of uris which have unencoded spaces
      (lp: #313686)

 -- Sebastien Bacher <email address hidden> Fri, 17 Apr 2009 11:12:16 +0200

Changed in libsoup2.4 (Ubuntu Jaunty):
status: Fix Committed → Fix Released
Revision history for this message
Martin Pitt (pitti) wrote :

Seb says this is fixed in karmic.

Changed in libsoup2.4 (Ubuntu):
status: Fix Committed → Fix Released
Changed in totem:
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

Other bug subscribers

Remote bug watches

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