Shouldn't escape reserved characters

Bug #502488 reported by Guillaume Desmottes
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Moovida
New
Undecided
Unassigned

Bug Description

As reported on bug #500445 there is currently an issue with the ARTE plugin. After investigation, it seems that's because of this commit:
http://bazaar.launchpad.net/~moovida-developers/moovida/relook/revision/1583.1.1

(That's weird because I'm pretty sure the plugin was working with recent versions of Moovida; maybe they change something in their servers).

Video URIs are of the form:
mms://artestras.wmod.llnwd.net/a3903/o35/geo/arte7/default/arteprod/A7_SGT_ENC_07_038933-004-A_PG_HQ_FR.wmv?e=1262480886&h=c4b94199a6b3099d99f763d7b697ba8c
which are now escaped to:
mms://artestras.wmod.llnwd.net/a3903/o35/geo/arte7/default/arteprod/A7_SGT_ENC_07_038933-004-A_PG_HQ_FR.wmv%3Fe%3D1262480886%26h%3Dc4b94199a6b3099d99f763d7b697ba8c

The former is accepted by the server, the latter is not.

According to my quick reading of http://www.ietf.org/rfc/rfc2396.txt section 2.2, reserved characters shouldn't be escaped when they are used as parameters delimiters.

An easy fix for this bug is to add those characters to the unescape list passed to the quote function.

Related branches

Revision history for this message
Guillaume Desmottes (cassidy) wrote :

I pushed a fix to the attached branch.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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