[#DISPL-246] Wrong header set in BufferedResponseWrapper when a charset is already defined (missing '=') - jira.codehaus.org

DisplayTag

Wrong header set in BufferedResponseWrapper when a charset is already defined (missing '=')

Created: 29/Sep/05 07:53 PM Updated: 11/Dec/06 09:07 AM Resolved: 26/Oct/05 03:54 PM

Return to search

Component/s:

Build/distribution

Affects Version/s:

1.0

Fix Version/s:

1.1

Time Tracking:

Not Specified

Application server:

none

Description

« Hide

In BufferedResponseWrapper13Impl you set the content type and i think you have forgotten '='.
In source 1.0 code is:
BufferedResponseWrapper13Impl
getResponse().setContentType(StringUtils.substringBefore(theContentType, "charset") + ' ' + charset);
and i think it should be:
getResponse().setContentType(StringUtils.substringBefore(theContentType, "charset") + '=' + charset);

(see on '=' fragment).
Best regards.

Sort Order:

fabrizio giustina made changes - 26/Oct/05 03:53 PM

Field	Original Value	New Value
Summary	BufferedResponseWrapper13Impl	Wrong header set in BufferedResponseWrapper when a charset is already defined (missing '=')

[ Permalink | « Hide ]

fabrizio giustina added a comment - 26/Oct/05 03:54 PM

fixed in CVS for 1.1, thanks

fabrizio giustina made changes - 26/Oct/05 03:54 PM

Status	Open [ 1 ]	Closed [ 6 ]
Fix Version/s		1.1 [ 11225 ]
Resolution		Fixed [ 1 ]