|     protected boolean matchRequest(Request request) {
  | 
  |       // Has a session been created?
  |       Session session = request.getSessionInternal(false);
  |       if (session == null)
  |           return (false);
  | 
  |       // Is there a saved request?
  |       SavedRequest sreq = (SavedRequest)
  |           session.getNote(Constants.FORM_REQUEST_NOTE);
  |       if (sreq == null)
  |           return (false);
  | 
  |       // Is there a saved principal?
  |       if (session.getNote(Constants.FORM_PRINCIPAL_NOTE) == null)
  |           return (false);
  | 
  |       // Does the request URI match?
  |       String requestURI = request.getRequestURI();
  |       if (requestURI == null)
  |           return (false);
  |       return (requestURI.equals(request.getRequestURI()));
  | 
  |     }
  | 
surely should be 

  |     protected boolean matchRequest(Request request) {
  | 
  |       // Has a session been created?
  |       Session session = request.getSessionInternal(false);
  |       if (session == null)
  |           return (false);
  | 
  |       // Is there a saved request?
  |       SavedRequest sreq = (SavedRequest)
  |           session.getNote(Constants.FORM_REQUEST_NOTE);
  |       if (sreq == null)
  |           return (false);
  | 
  |       // Is there a saved principal?
  |       if (session.getNote(Constants.FORM_PRINCIPAL_NOTE) == null)
  |           return (false);
  | 
  |       // Does the request URI match?
  |       String requestURI = request.getRequestURI();
  |       if (requestURI == null)
  |           return (false);
  |       return (requestURI.equals(sreq.getRequestURI()));
  | 
  |     }
  | 
since as things stand they'll always match because the string is being compared 
to itself.

View the original post : 
http://www.jboss.com/index.html?module=bb&op=viewtopic&p=3947648#3947648

Reply to the post : 
http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=3947648


-------------------------------------------------------
All the advantages of Linux Managed Hosting--Without the Cost and Risk!
Fully trained technicians. The highest number of Red Hat certifications in
the hosting industry. Fanatical Support. Click to learn more
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=107521&bid=248729&dat=121642
_______________________________________________
JBoss-user mailing list
JBoss-user@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jboss-user

Reply via email to