[
https://issues.apache.org/jira/browse/FELIX-2150?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Karl Pauls updated FELIX-2150:
------------------------------
Fix Version/s: framework-3.0.0
Affects Version/s: framework-2.0.5
Include this in 3.0
> URLStreamHandlerProxy.setURL may not set query component correctly
> ------------------------------------------------------------------
>
> Key: FELIX-2150
> URL: https://issues.apache.org/jira/browse/FELIX-2150
> Project: Felix
> Issue Type: Bug
> Components: Framework
> Affects Versions: framework-2.0.4, framework-2.0.5
> Reporter: Sahoo
> Assignee: Karl Pauls
> Fix For: framework-3.0.0
>
>
> On Mon, Mar 1, 2010 at 12:56 PM, Sahoo <sa...@sun.com> wrote:
> > > Hi,
> > >
> > > org.apache.felix.framework.URLStreamHandlerProxy has following methods:
> > >
> > > public void setURL(
> > > URL url, String protocol, String host, int port, String authority,
> > > String userInfo, String path, String query, String ref)
> > > {
> > > super.setURL(url, protocol, host, port, authority, userInfo, path,
> > > query, ref);
> > > }
> > >
> > > public void setURL(
> > > URL url, String protocol, String host, int port, String file, String
> > > ref)
> > > {
> > > super.setURL(url, protocol, host, port, null, null, file, null,
> > > ref);
> > > }
> > >
> > >
> > > There appears to be a *bug* in the latter method. It passes "file" as
> > > "path." Should file not be brone into path and query components which
> > > would
> > > have automatically happened if
> > > super.setURL(url, protocol, host, port, file, ref) been called? Any
> > > comments? I have not done any testing, just concluding based on code
> > > inspection.
> I agree, looks like a bug. It is not as bad as the path can be the
> file as well but if you would call getQuery() on the resulting url it
> will return null i think (even if you had a query). Could you create a
> jira for this?
> Thanks and regards,
> Karl
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
