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

--- Comment #10 from Mark Thomas <ma...@apache.org> 2010-07-02 14:10:42 EDT ---
(In reply to comment #9)
> Any feedback on my second patch?  I've been using it in production for more
> than a month now with no obvious issues.  I realize that you guys are probably
> busy getting Tomcat 7 ready otherwise, but if you could take a look sometime
> soon, that would be great.

One suggested change. I think you should set the validator class name rather
than the validator directly so the entire config can be done in the resource.
Other than that, looks good.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- 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