31
  RestTemplate restTemplate = new RestTemplate();

  final MappingJackson2XmlHttpMessageConverter converter = new MappingJackson2XmlHttpMessageConverter();
  final List<MediaType> supportedMediaTypes = new LinkedList<MediaType>(converter.getSupportedMediaTypes());
  supportedMediaTypes.add(MediaType.ALL);
  converter.setSupportedMediaTypes(supportedMediaTypes);
  restTemplate.getMessageConverters().add(converter);  


  ResponseEntity<MyDTO[]> response = restTemplate.getForEntity(urlBase, MyDTO[].class);

  HttpHeaders headers = response.getHeaders();
  URI location = headers.getLocation(); // Has my redirect URI

  response.getBody(); //Always null

I was under the impression that a 302 would automatically be followed. Am I incorrect in this assumption? I now need to pick off this location and re-request?

techie.brandon
  • 1,638
  • 2
  • 18
  • 27

4 Answers4

25

Using the default ClientHttpRequestFactory implementation - which is the SimpleClientHttpRequestFactory - the default behaviour is to follow the URL of the location header (for responses with status codes 3xx) - but only if the initial request was a GETrequest.

Details can be found in this class - searching for the following method:

protected void prepareConnection(HttpURLConnection connection, String httpMethod) throws IOException {

    ...

    if ("GET".equals(httpMethod)) {
        connection.setInstanceFollowRedirects(true);
    }

Here the relevant doc comment of HttpURLConnection.setInstanceFollowRedirects method:

Sets whether HTTP redirects (requests with response code 3xx) should be automatically followed by this {@code HttpURLConnection} instance.

The default value comes from followRedirects, which defaults to true.

fateddy
  • 6,887
  • 3
  • 22
  • 26
  • Provided that I am using getForEntity it would be assumed a GET request. I went ahead and ensured the use of the SimpleClientHttpRequestFactory in the restTemplate with only the same result. Going to step into the code you highlighted and see what is happening. – techie.brandon Apr 02 '15 at 21:55
  • Because your response contains the `location` header - it tells you that the redirect did not happen. Otherwise the response would contain the result of the very last response received. The only thing could be that the response status is not a `3xx` - the location header is ignored. Check the response status code (`response.getStatusCode()`) first. – fateddy Apr 03 '15 at 02:12
  • 16
    I did `new RestTemplate(new SimpleClientHttpRequestFactory() { protected void prepareConnection(HttpURLConnection connection, String httpMethod) throws IOException { super.prepareConnection(connection, httpMethod); connection.setInstanceFollowRedirects(false); } });` to disable redirection – mzzzzb Apr 07 '16 at 11:20
  • It follows only one redirect – Skeeve Nov 09 '18 at 14:44
11

Are you trying to redirect from one protocol to another, e.g. from http to https or vise versa? If so the automatic redirect won't work. See this comment: URLConnection Doesn't Follow Redirect

After discussion among Java Networking engineers, it is felt that we shouldn't automatically follow redirect from one protocol to another, for instance, from http to https and vise versa, doing so may have serious security consequences

from https://bugs.java.com/bugdatabase/view_bug.do?bug_id=4620571

Otherwise if you debug the RestTemplate code you will see that by default HttpURLConnection is set properly with getInstanceFollowRedirects() == true.

Vladimir Mitev
  • 431
  • 6
  • 8
2

When using the CommonsClientHttpRequestFactory (which uses HttpClient v3 underneath) you can override the postProcessCommonsHttpMethod method and set to follow redirects.

public class FollowRedirectsCommonsClientHttpRequestFactory extends CommonsClientHttpRequestFactory {

  @Override
  protected void postProcessCommonsHttpMethod(HttpMethodBase httpMethod) {
    httpMethod.setFollowRedirects(true);
  }
}

You can then use it like this (with optional, possibly preconfigured, HttpClient instance) and requests will follow the location headers in response:

RestTemplate restTemplate = new RestTemplate(
      new FollowRedirectsCommonsClientHttpRequestFactory());
Mrkvozrout
  • 314
  • 1
  • 9
-3

Try create RestTemplate like this (Spring config):

@Bean("smartRestTemplate")
public RestTemplate restTemplate(RestTemplateBuilder restTemplateBuilder) {
    return restTemplateBuilder
            .setConnectTimeout(Duration.ofSeconds(..))
            .setReadTimeout(Duration.ofSeconds(..))
            .build();
}
Grigory Kislin
  • 16,647
  • 10
  • 125
  • 197