Mark St.Godard wrote:

Hi Ben,

The code snippet I sent a few weeks back did work on both Tomcat and
Websphere 5 for me.

http://www.mail-archive.com/[EMAIL PROTECTED]/ms
g00118.html


Essentially using something like this in AbtractProcessingFilter's doFilter()

if ( httpRequest.getRequestURL().toString().endsWith(
       httpRequest.getContextPath() + filterProcessesUrl) )


instead of : if (filterProcessesUrl.equals(httpRequest.getServletPath())) {


I personally make use heavy use of Websphere (as well as the occasional Tomcat)

and I think the proper thing to do would be to provide a solution that works
and satisfies all of the integration tests

and I think the above code will suffice.

Again, filters are a different beast... and hence the grey area.

I just dont want to get into the servlet container spec wars (Jetty vs.
Websphere) and have a mutually exclusive solution.

My 2 cents

Colin any thoughts?


Hi Mark

Your solution passed the Acegi Security automated integration tests for Jetty, Catalina and JBoss as well, so I've committed this change to CVS.

Best regards
Ben



-------------------------------------------------------
This SF.Net email sponsored by Black Hat Briefings & Training.
Attend Black Hat Briefings & Training, Las Vegas July 24-29 - digital self defense, top technical experts, no vendor pitches, unmatched networking opportunities. Visit www.blackhat.com
_______________________________________________
Acegisecurity-developer mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/acegisecurity-developer

Reply via email to