hc-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michiel Proce (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HTTPCLIENT-1294) CircularRedirectException is falsely thrown on URI case mismatch
Date Tue, 08 Jan 2013 14:24:12 GMT

    [ https://issues.apache.org/jira/browse/HTTPCLIENT-1294?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13546914#comment-13546914
] 

Michiel Proce commented on HTTPCLIENT-1294:
-------------------------------------------

Workaround, in own implementation of org.apache.http.impl.client.DefaultRedirectStrategy:

{code:java}
    @Override
    protected URI createLocationURI(String location) throws ProtocolException {
          // Let HttpClient build an URI
        URI locationURI = super.createLocationURI(location);

        // Workaround HttpClient bug: https://issues.apache.org/jira/browse/HTTPCLIENT-1294
        // Lower case the host (case insenistive according to RFC-3986 http://tools.ietf.org/html/rfc3986#section-3.1)
        // This way we prevent an extra redirect from IIS6: EXAMPLE.com -> example.com.
These hosts are considered equal, and throw a CircularRedirectException
        if (locationURI != null) {
            String host = locationURI.getHost();
            if (host != null) {
                String hostLowerCase = host.toLowerCase();
                // if the host contains uppercase characters
                if (!host.equals(hostLowerCase)) {
                    // rebuild the URI with lower case host
                    try {
                        locationURI = new URIBuilder(locationURI)
                                .setHost(host.toLowerCase())
                                .build();
                    } catch (URISyntaxException ex) {
                        throw new ProtocolException("Invalid redirect URI: " + location, ex);
                    }
                }
            }
        }

        return locationURI;
    }
{/code}
                
> CircularRedirectException is falsely thrown on URI case mismatch
> ----------------------------------------------------------------
>
>                 Key: HTTPCLIENT-1294
>                 URL: https://issues.apache.org/jira/browse/HTTPCLIENT-1294
>             Project: HttpComponents HttpClient
>          Issue Type: Bug
>          Components: HttpClient
>    Affects Versions: 4.1.3, 4.2.2
>            Reporter: Michiel Proce
>
> Some servers (including IIS6) redirect wrong-cased request URIs to lower case:
> http://EXAMPLE.com/ -> http://example.com/
> Now when I'm redirected from another URI to an uppercase URI, the following redirects
happen:
> http://referrer.com/ -> http://EXAMPLE.com/ -> http://example.com/
> When running this request with HttpClient (with default ALLOW_CIRCULAR_REDIRECTS: false),
I get a org.apache.http.client.CircularRedirectException, even though a circular redirect
won't occur.
> The problem lies in java.net.URI, the following URIs are considered equal:
> URI a = new URI("http://example.com");
> URI b = new URI("http://EXAMPLE.com");
> // a.equals(b): true
> // a.hashCode() == b.hashCode(): true
> The redirect locations are stored in a HashSet<URI> in org.apache.http.impl.client.RedirectLocations.
A CircularRedirectException is thrown when an URI is already in this hashset.
> I have no real suggestions on how to fix this, because I am not known with the coding
style for HttpClient..

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
For additional commands, e-mail: dev-help@hc.apache.org


Mime
View raw message