On Mon, Jul 30, 2007 at 02:41:17PM +0200, Paul Sandoz wrote:
> >Attaching a patch for the "URI in URI case", but I think
> >it is not the right way to go.
> >
>
> Hmm... not sure. We need to make this independent of the container and
> my gut feeling is the ensureStringIsPartOfURI is a bit of a hack.
+1, it is a hack, but without such a hack an exception is thrown.
I am going to file a bug report for getPathInfo method.
>
>
> 1) Change isRedirectToNormalizedURI to be isRedirectToCanonicalURI
> and clearly specify this as being URI normalization + changing '/+'
> to '/'; and
>
> 2) Specify an APT processing option to switch redirection off.
>
> it might make sense to split 1 into two options: normalization; and
> normalization with '/+' -> '/'.
+1 for having the 2 options. Can you give me some guidance,
how to implement?
(I see isRedirectToNormalizedURI is hardcoded in ResourceConfigImpl
in ContainerFactory).
Thanks,
~Jakub
>
> Paul.
>
> >~Jakub
> >
> >>Paul.
> >>
> >>[1] http://gbiv.com/protocols/uri/rfc/rfc3986.html#relative-dot-segments
> >>
> >>>The following code:
> >>>--cuthere--
> >>>URI baseURI = new URI("http://host/path1//path2");
> >>>System.out.println("baseURI.toString() = " + baseURI.toString());
> >>>System.out.println("baseURI.normalize().toString() = " +
> >>>baseURI.normalize().toString());
> >>>--cuthere--
> >>>
> >>>Generates:
> >>>--cuthere--
> >>>baseURI.toString() = http://host/path1//path2
> >>>baseURI.normalize().toString() = http://host/path1/path2
> >>>--cuthere--
> >>>
> >>>~Jakub
> >>>
> >>>
> >>>On Thu, Jul 26, 2007 at 01:19:43PM -0400, Marc Hadley wrote:
> >>>>According to RFC 3986[1], section 3.3 the : is allowed in a path
> >>>>segment so there shouldn't be any reason to encode it. In addition I
> >>>>don't see anything about removing double '/' characters so I think
> >>>>that getPathInfo has a bug.
> >>>>
> >>>>Marc.
> >>>>
> >>>>[1] http://ietf.org/rfc/rfc3986.txt
> >>>>
> >>>>On Jul 25, 2007, at 9:03 AM, Jakub Podlesak wrote:
> >>>>
> >>>>>Hi all,
> >>>>>
> >>>>>I have tried following url:
> >>>>>
> >>>>>http://localhost:8080/Bookmark/resources/users/japod/bookmarks/
> >>>>>ftp://any.net/file.txt
> >>>>>
> >>>>>and noticed, that corresponding HttpServletRequest (tested on
> >>>>>glassfish) provides:
> >>>>>
> >>>>>getRequestURI() ->"/Bookmark/resources/users/japod/bookmarks/ftp://
> >>>>>any.net/file.txt"
> >>>>>getPathInfo()->"/users/japod/bookmarks/ftp:/any.net/file.txt"
> >>>>> ^^^^^^^^
> >>>>>
> >>>>>Please note the missing slash in the latter (ftp:/any.net instead
> >>>>>of ftp://any.net)
> >>>>>
> >>>>>Is this a bug or a feature?
> >>>>>
> >>>>>~Jakub
> >>>>>
> >>>>>
> >>>>>P.S. It causes an exception in [setURIs] method of
> >>>>>[com.sun.ws.rest.impl.container.servlet.HttpRequestAdaptor]
> >>>>---
> >>>>Marc Hadley <marc.hadley at sun.com>
> >>>>CTO Office, Sun Microsystems.
> >>>>
> >>>>
> >>--
> >>| ? + ? = To question
> >>----------------\
> >> Paul Sandoz
> >> x38109
> >>+33-4-76188109
> >>
> >>---------------------------------------------------------------------
> >>To unsubscribe, e-mail: dev-unsubscribe_at_jersey.dev.java.net
> >>For additional commands, e-mail: dev-help_at_jersey.dev.java.net
> >>
> >>
> >>------------------------------------------------------------------------
> >>
> >>---------------------------------------------------------------------
> >>To unsubscribe, e-mail: dev-unsubscribe_at_jersey.dev.java.net
> >>For additional commands, e-mail: dev-help_at_jersey.dev.java.net
>
> --
> | ? + ? = To question
> ----------------\
> Paul Sandoz
> x38109
> +33-4-76188109
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe_at_jersey.dev.java.net
> For additional commands, e-mail: dev-help_at_jersey.dev.java.net
>