struts-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jose L Martinez-Avial (JIRA)" <>
Subject [jira] [Commented] (WW-4311) HttpHeaderResult should allow to provide a body for the response
Date Wed, 26 Mar 2014 02:21:15 GMT


Jose L Martinez-Avial commented on WW-4311:

Actually, taking a look at the code. I think we also need to change the status to be an String,instead
of an int, and make it parseable. This way it would behave the same way as the error/errorMessage
fields. The main difference I see is that there is a constructor HttpHeaderResult(int status),
and it would be impacted. Is that constructor used somewhere?

> HttpHeaderResult should allow to provide a body for the response
> ----------------------------------------------------------------
>                 Key: WW-4311
>                 URL:
>             Project: Struts 2
>          Issue Type: Improvement
>          Components: Dispatch Filter
>            Reporter: Jose L Martinez-Avial
>            Priority: Minor
> Curently the HttpHeaderResult can only write to the body response if it is the result
has a parameter error (through the errorMessage setter), and then it uses javax.servlet.http.HttpServletResponse#sendError(int,
String) to send the error, with the error message in the body, to the client.
> If the result has a parameters status, and no error, then nothing is written to the response
body. It only set the status and the headers, and send it to the client.
> I suggest to add a parameter message that can be written to the response when using the
parameter status.
> I'm working on a patch for this, it shouldn't take long.

This message was sent by Atlassian JIRA

View raw message