[rfc][icedtea-web] Archive URL wrongly resolved - PR1204

Andrew Azores aazores at redhat.com
Fri Jun 14 06:45:41 PDT 2013


Changelog:

* netx/net/sourceforge/jnlp/cache/ResourceUrlCreator.java 
(getVersionedUrlUsingQuery): retain query strings in URLs
* plugin/icedteanp/java/sun/applet/PluginAppletViewer.java 
(requestPluginProxyInfo): do not add '/' character between hostname and 
path if the path is absolute

* 
tests/reproducers/simple/AbsolutePathAndQueryStringApplet/testcases/AbsolutePathAndQueryStringAppletTest.java: 
tests in-browser launching of applets with absolute JAR paths and query 
strings in HTML embed tags
* 
tests/reproducers/simple/AbsolutePathAndQueryStringApplet/srcs/AbsolutePathAndQueryStringApplet.java: 
same
* 
tests/reproducers/simple/AbsolutePathAndQueryStringApplet/resources/AbsolutePathAndQueryStringApplet.html: 
same

This patch fixes a bug where archive URLs would be incorrectly resolved 
in certain cases. If the archive path specified in an HTML Embed tag was 
of the form "/foo/bar.jar?i=1234abcd", on host some.site.com, then the 
plugin would attempt to resolve "http://some.site.com//foo/bar.jar". 
This was incorrect due to the doubled '/' character after the hostname 
as well as the removal of the query string.

Thanks,

Andrew
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fix.patch
Type: text/x-patch
Size: 3844 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20130614/bacbe652/fix.patch 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: reproducer.patch
Type: text/x-patch
Size: 7933 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20130614/bacbe652/reproducer.patch 


More information about the distro-pkg-dev mailing list