Semi-colon should not be encoded in URL

gcode-mirror / earth-api-samples

Automatically exported from code.google.com/p/earth-api-samples

0 stars 0 forks source link

Semi-colon should not be encoded in URL #106

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

What steps will reproduce the problem?
1. KmlLink.setUrl("http://localhost/foo;bar=2")
2. Click the resulting link
3. Observe that the semi-colon is wrongly encoded

What is the expected output or behavior? What do you see instead?

The semi-colon is part of the reserved set of characters for URLs and
should not be encoded.

Which plugin version are you using?

The latest one

Which browsers and operating systems are affected?

All

Please provide any additional information (code snippets/links) below.

Discussion:

http://groups.google.com/group/google-earth-browser-plugin/browse_thread/thread/
f0e44f23b87c14b9/ba2f772194a24813?lnk=gst&q=fromage#

Original issue reported on code.google.com by stephane...@gmail.com on 3 Dec 2008 at 10:45

GoogleCodeExporter commented 9 years ago

Hi, does this problem persist in the new version (4.3.11498.3261)? From the bug
report date it looks like you may have still been using 4.3.9685.8774.

Original comment by api.roman.public@gmail.com on 16 Dec 2008 at 1:14

GoogleCodeExporter commented 9 years ago

Version: 4.3.11528.8566

Using the sample:

http://earth-api-samples.googlecode.com/svn/trunk/examples/kml-fetch-interactive
.html

fetch the kml: 
http://earth-api-samples.googlecode.com/svn/trunk/examples/static/red.kml?foo=ba
r
OK

fetch the kml:
http://earth-api-samples.googlecode.com/svn/trunk/examples/static/red.kml;foo=ba
r
Not found

Look at HTTP headers in Proxy: 
GET /svn/trunk/examples/static/red.kml?foo=bar HTTP/1.1
GET /svn/trunk/examples/static/red.kml%3Bfoo%3Dbar HTTP/1.1

GE should not be encoding ';' and '=' in second URL.

Original comment by chris.p....@gmail.com on 16 Jan 2009 at 6:56

GoogleCodeExporter commented 9 years ago

Original comment by api.roman.public@gmail.com on 22 May 2009 at 1:52

Added labels: Component-API

GoogleCodeExporter commented 9 years ago

Original comment by jli...@google.com on 17 Mar 2010 at 6:16

Changed state: Acknowledged
Added labels: Internal-2522724
Removed labels: Component-API

GoogleCodeExporter commented 9 years ago

fixed in 5.2.0.5932

Original comment by bcke...@google.com on 11 Mar 2011 at 11:25

Changed state: Fixed