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

[Andrew Azores]

On 09/18/2013 02:26 AM, Jiri Vanek wrote:
>
>> (snip)
> ping?

Changelog:
* netx/net/sourceforge/jnlp/cache/ResourceUrlCreator.java: 
(getVersionedUrlUsingQuery) renamed to getVersionedUrl, refactored 
construction of URL
* plugin/icedteanp/java/sun/applet/PluginAppletViewer.java: 
(requestPluginProxyInfo) extracted proxy URI logic. (processProxyUri) 
new method for finding proxy URIs, handles absolute resource paths correctly
* 
tests/netx/unit/net/sourceforge/jnlp/cache/ResourceUrlCreatorTest.java: 
added tests for ResourceUrlCreator#getVersionedUrl
* tests/netx/unit/sun/applet/PluginAppletViewerTest.java: added tests 
for PluginAppletViewer.processProxyUri
* 
tests/reproducers/simple/AbsolutePathsAndQueryStrings/testcase/AbsolutePathsAndQueryStrings.java: 
new reproducer checks that absolute paths and query strings in resource 
URLs are properly handled, and caching still works
* 
tests/reproducers/simple/AbsolutePathsAndQueryStrings/resource/AbsolutePathsAndQueryStrings.html: 
same
* 
tests/reproducers/simple/AbsolutePathsAndQueryStrings/resource/AbsolutePathsAndQueryStrings.jnlp: 
same

Here's a new version of this change set, now with more testing! The 
actual fix is the same. The reproducer is a little bit lame but I think 
I captured most of the cases with the unit tests anyway already.

Thanks,

-- 
Andrew A

-------------- next part --------------
A non-text attachment was scrubbed...
Name: fix.patch
Type: text/x-patch
Size: 5154 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20130925/5a8d57e1/fix-0001.patch 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: reproducer.patch
Type: text/x-patch
Size: 10717 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20130925/5a8d57e1/reproducer-0001.patch 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: unit_tests.patch
Type: text/x-patch
Size: 5882 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20130925/5a8d57e1/unit_tests-0001.patch