Johannes Koch schrieb:
Ortwin Glück schrieb:
Yep, what about the fragment component?

No, fragments are not handled by HTTP Servers. They are only handeled by clients (browser).

Then to make it consistent, I will also change

URI absoluteRequestURI = new URI(
        target.getSchemeName(),
        null,
        target.getHostName(),
        target.getPort(),
        requestURI.getPath(),
        requestURI.getQuery(),
        requestURI.getFragment());

to
...
        requestURI.getQuery(),
        null);


See attached patch.

--
Johannes Koch
BIKA Web Compliance Center - Fraunhofer FIT
Schloss Birlinghoven, D-53757 Sankt Augustin, Germany
Phone: +49-2241-142628    Fax: +49-2241-142065
Index: 
D:/koch/opt/eclipse/workspace_p/httpclient_module-client/src/main/java/org/apache/http/impl/client/DefaultRedirectHandler.java
===================================================================
--- 
D:/koch/opt/eclipse/workspace_p/httpclient_module-client/src/main/java/org/apache/http/impl/client/DefaultRedirectHandler.java
      (revision 610764)
+++ 
D:/koch/opt/eclipse/workspace_p/httpclient_module-client/src/main/java/org/apache/http/impl/client/DefaultRedirectHandler.java
      (working copy)
@@ -143,7 +143,7 @@
                         target.getPort(),
                         requestURI.getPath(),
                         requestURI.getQuery(),
-                        requestURI.getFragment());
+                        null);
                 uri = absoluteRequestURI.resolve(uri); 
             } catch (URISyntaxException ex) {
                 throw new ProtocolException(ex.getMessage(), ex);
@@ -169,7 +169,7 @@
                             uri.getHost(),
                             uri.getPort(),
                             uri.getPath(),
-                            null,
+                            uri.getQuery(),
                             null);
                 } catch (URISyntaxException ex) {
                     throw new ProtocolException(ex.getMessage(), ex);
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to