2013/10/18 Dave Newton <davelnew...@gmail.com>: > On Fri, Oct 18, 2013 at 8:01 AM, Lukasz Lenart wrote: > >> Also changed order of methods in acceptableName() method to first >> check "excludeParams" and then "acceptParams" >> > > Meaning if it's in the exclude list it doesn't check accepted?
Yes, boolean accepted = isWithinLengthLimit(name) && !isExcluded(name) && isAccepted(name); The problem is with "method:" and "action:" prefixes which are added to excludedParams list but doesn't match ACCEPTED_PARAM_NAMES and then in the logs user see warning: "Parameter [method:index] didn't match acceptedPattern pattern!" which is a bit misleading. After switch, user will see: "Parameter [method:index] is on the excludeParams list of patterns!" which has more sense. Regards -- Łukasz + 48 606 323 122 http://www.lenart.org.pl/ --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org
