HttpClientErrorException should provide access to response headers [SPR-7938]

[spring-projects-issues]

Julian Reschke opened SPR-7938 and commented

It would be good if HttpClientErrorException would not only provide access to the response body but also to response headers,

---

Sub-tasks:

• Backport "HttpClientErrorException should provide access to response headers" [SPR-9460] #14095 Backport "HttpClientErrorException should provide access to response headers"

Referenced from: commits bca2357, c471bdd, b992c3d

2 votes, 5 watchers

[spring-projects-issues]

Arjen Poutsma commented

Moved to SPR.

[spring-projects-issues]

Scott Andrews commented

#86

[spring-projects-issues]

Dave Syer commented

I'm not sure if anyone tested this in a live system, but there is likely to be a nasty problem waiting for anyone using it out of the box. I think that the default implementation of HttpClientResponse (the JDK one) barfs if you try and get the headers from an error response, or even a non-OK one in my experience. It might be a good idea to be defensive about this line:

HttpHeaders headers = response.getHeaders();

Wrap it in a try/catch or something?

[spring-projects-issues]

Chris Beams commented

Thanks for the heads-up, Dave. Scott, can you follow up and look into this?

[spring-projects-issues]

Scott Andrews commented

@Dave The change set didn't change the semantics on when or how the HttpHeaders are accessed. If there is an issue, it a preexisting bug.

[spring-projects-issues]

Rossen Stoyanchev commented

If I'm not mistaken the issue is specifically for a 401 response? #14004