[jboss-jira] [JBoss JIRA] Commented: (JBWEB-67) Patch to fix usabillity issues with the rewrite valve

[ http://jira.jboss.com/jira/browse/JBWEB-67?page=comments#action_12340765 ] Rob Hughes commented on JBWEB-67: --------------------------------- I am unsure where to have a discussion about this if you don't want the report reopened, so I won't reopen it but am adding this comment. I guess I have misstated what the problem is due to the fact that the example I provided had an issue with the RewriteCond and the query string and not the rule itself. With mod_rewrite and the rewrite valve the pattern is always matched against the full TestString. I am not disagreeing with that statement. What I am disagreeing with is by using Matcher.matches() you force the pattern to match the entire TestString. There is a subtle difference between matching the pattern to the entire string and forcing the pattern to match the entire string. Mod_rewrite does not do this, in the documentation it states that the URL or TestString is matched against the pattern. Nowhere does it state that the pattern must match the entire string and in actual usage you will find that it doesn't have to. For Example Given RewriteCond %{QUERY_STRING} name=([^&;]*) and Input name=about&test=param Mod_rewrite will process this condition as true while the current rewrite valve will process it as false. I would have to change the pattern to be .*name=([^&;]*).* to get it to match. If you are not aiming to provide a rewrite module that functions similar to mod_rewrite that this is a moot point and you can leave it as is, but it should be documented to state that the pattern must match the TestString in it's entirety. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira