You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hc.apache.org by ol...@apache.org on 2008/04/23 20:29:06 UTC

svn commit: r650988 - /httpcomponents/httpclient/trunk/module-client/src/main/java/org/apache/http/impl/client/DefaultRedirectHandler.java

Author: olegk
Date: Wed Apr 23 11:29:05 2008
New Revision: 650988

URL: http://svn.apache.org/viewvc?rev=650988&view=rev
Log:
Rewrite the redirect URI only if it contains a fragment part

Modified:
    httpcomponents/httpclient/trunk/module-client/src/main/java/org/apache/http/impl/client/DefaultRedirectHandler.java

Modified: httpcomponents/httpclient/trunk/module-client/src/main/java/org/apache/http/impl/client/DefaultRedirectHandler.java
URL: http://svn.apache.org/viewvc/httpcomponents/httpclient/trunk/module-client/src/main/java/org/apache/http/impl/client/DefaultRedirectHandler.java?rev=650988&r1=650987&r2=650988&view=diff
==============================================================================
--- httpcomponents/httpclient/trunk/module-client/src/main/java/org/apache/http/impl/client/DefaultRedirectHandler.java (original)
+++ httpcomponents/httpclient/trunk/module-client/src/main/java/org/apache/http/impl/client/DefaultRedirectHandler.java Wed Apr 23 11:29:05 2008
@@ -155,7 +155,7 @@
             }
             
             URI redirectURI;
-            if (uri.getQuery() != null || uri.getFragment() != null) {
+            if (uri.getFragment() != null) {
                 try {
                     HttpHost target = new HttpHost(
                             uri.getHost(),