tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mark Thomas <ma...@apache.org>
Subject Re: svn commit: r1156530 - in /tomcat/trunk/java/org/apache/catalina/connector: LocalStrings.properties Response.java
Date Thu, 11 Aug 2011 11:13:50 GMT
On 11/08/2011 11:53, Tim Funk wrote:
> It may be overkill ... but should "absolute" be XML escaped?

I don't think so. It is meant to be a valid, encoded URL. It it isn't,
it will cause problems in the location header too.

Mark

> 
> -Tim
> 
> 
> On Thu, Aug 11, 2011 at 4:13 AM, <markt@apache.org> wrote:
> 
>> Modified: tomcat/trunk/java/org/apache/catalina/connector/Response.java
>> URL:
>> http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/connector/Response.java?rev=1156530&r1=1156529&r2=1156530&view=diff
>>
>> ==============================================================================
>> +            PrintWriter writer = getWriter();
>> +            writer.print(
>> +                    sm.getString("coyoteResponse.sendRedirect.note",
>> absolute));
>> +            flushBuffer();
>>         } catch (IllegalArgumentException e) {
>>             setStatus(SC_NOT_FOUND);
>>         }
>>
>>
> 


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org


Mime
View raw message