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: felix-2.0.4
Reporter: Sahoo
On Mon, Mar 1, 2010 at 12:56 PM, Sahoo <[email protected]> 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.
