Mailing-List: contact dev-help@hc.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "HttpComponents Project" <dev@hc.apache.org>
Date: Tue, 17 May 2011 10:35:47 +0000 (UTC)
From: "Oleg Kalnichevski (JIRA)" <jira@apache.org>
To: dev@hc.apache.org
Message-ID: 
 <2100721458.18749.1305628547553.JavaMail.tomcat@hel.zones.apache.org>
In-Reply-To: 
 <627738875.10804.1305310547379.JavaMail.tomcat@hel.zones.apache.org>
Subject: [jira] [Commented] (HTTPCLIENT-1089) HTTP Client does not handle
 302 redirect in POST request
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


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

Oleg Kalnichevski commented on HTTPCLIENT-1089:
-----------------------------------------------

File committed to SVN trunk as LaxRedirectStrategy [1]. Many thanks for this contribution, Bartosz. Some test coverage for it would be great, though.

[1] http://svn.apache.org/viewvc?rev=1104115&view=rev

> HTTP Client does not handle 302 redirect in POST request
> --------------------------------------------------------
>
>                 Key: HTTPCLIENT-1089
>                 URL: https://issues.apache.org/jira/browse/HTTPCLIENT-1089
>             Project: HttpComponents HttpClient
>          Issue Type: Bug
>          Components: HttpClient
>    Affects Versions: 4.1.1
>         Environment: *
>            Reporter: Bartosz Firyn
>              Labels: redirect
>         Attachments: BrowserRedirectStrategy.java
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> POST requests are not being automatically redirected after receiving 302 response.
> This is because there is a statement in the DefaultRedirectStrategy class:
> 82:        case HttpStatus.SC_MOVED_TEMPORARILY:
> 83:            return (method.equalsIgnoreCase(HttpGet.METHOD_NAME)
> 84:                || method.equalsIgnoreCase(HttpHead.METHOD_NAME)) && locationHeader != null;
> Which means that only HEAD and GET requests are being redirected. In my opinion all requests should be redirected, not only these two types.
> This should be:
> 82:        case HttpStatus.SC_MOVED_TEMPORARILY:
> 83:            return locationHeader != null;
> After receiving 302 status for POST request new GET request should be send to Location URI. Internet browsers works in the same way.
> Best Regards
> Bartosz Firyn

--
This message is automatically generated by JIRA.
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