https://issues.apache.org/bugzilla/show_bug.cgi?id=56265

--- Comment #6 from Mark Thomas <ma...@apache.org> ---
Created attachment 31401
  --> https://issues.apache.org/bugzilla/attachment.cgi?id=31401&action=edit
Proposed patch v3

I've been looking at v2 of the patch.

I think I found a couple of bugs in the testBug56265()
- The method wasn't annotated with @Test so the test did not execute
- bug56265.jsp assigned a value to a request attribute named 'text' but tried
to use an attribute named 'world'.
- The test was expected value [1] to be unescaped but I think it should have
been expecting it to be escaped

I've attached v3 of the patch for comment that - assuming my analysis of v2 is
correct - addresses the issues I found and adds a few more tests. The fix part
of the patch is now a lot simpler.

-- 
You are receiving this mail because:
You are the assignee for the bug.

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

Reply via email to