yasserzamani commented on code in PR #559:
URL: https://github.com/apache/struts/pull/559#discussion_r891886189
##########
core/src/main/java/com/opensymphony/xwork2/interceptor/ParametersInterceptor.java:
##########
@@ -354,7 +407,86 @@ protected boolean isExcluded(String paramName) {
}
return false;
}
+
+ public void setAcceptedValuePatterns(String commaDelimitedPatterns) {
Review Comment:
Hmm I think it makes sense to have some default patterns set here however.
Currently it's empty and user has to set it in all interceptors stacks as far
as I understood right? If so then like what is done for parameters names,
better to have some defaults patterns set here e.g. your already provided
patterns in tests for %{} and ${}
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
