In order to handle non standard HTTP Status codes returned by services in RestTemplate, I would like to add support for handling non standard status codes and passing on responses for these responses to callers [SPR-9311]

[spring-projects-issues]

harish opened SPR-9311 and commented

Currently Resttemplate uses an enum of 'standard' HTTP status codes, that is subset of all legally possible HTTP status codes that services can return. When a non standard HTTP status code is returned by services, an IllegalArgumentException is thrown and all response body is ignored. We call services that use custom code 450 for user validation errors along with an xml containing list of messages codes and message text. Unfortunately we get only an illegal argument exception and response body is lost. I think it would be helpful if say all 4xx, 1xx, 2xx, 5xx are treated as legal responses and their response bodies are preserved and passed to the caller. At the least a hook could be provided to extend the enum used by Resttemplate to add additional HTTP codes that caller might expect.

---

Issue Links:

• RestTemplate throws IllegalArgumentException when HTTP status is not in the HttpStatus enum [SPR-6752] #11418 RestTemplate throws IllegalArgumentException when HTTP status is not in the HttpStatus enum ("duplicates")

2 votes, 3 watchers

[spring-projects-issues]

harish commented

This is blocking us from using RestTemplate. Is there a way to bump up the priority?

[spring-projects-issues]

Arjen Poutsma commented

I believe the functionality you seek is already there, see #11418

[spring-projects-issues]

harish commented

Thanks Arjen. Using a custom error handler gets us past this, but we find that we get errors at other places (where ever HttpsStatus is accessed internally by resttenplate). Seems it's tightly coupled to HttpStatus Enum. Since all 1xx,2xx,3xx, 4xx and 5xx are legal, it seems natural that HttpStatus enum include them or not to throw illegalStateException when passed. I think treating all these codes as legal by HttpStatus enum will help us a lot.