Skip to content

Commit

Permalink
Added the same warning that appears in the ParametersInterceptor abou…
Browse files Browse the repository at this point in the history
…t using ParameterNameAware to the JavaDoc for this interface

git-svn-id: https://svn.apache.org/repos/asf/struts/struts2/trunk@1508075 13f79535-47bb-0310-9956-ffa450edef68
  • Loading branch information
Bruce Allen Phillips committed Jul 29, 2013
1 parent 6cfc28b commit 49d339f
Showing 1 changed file with 7 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,13 @@
* ParametersInterceptor}. For example, actions may want to create a whitelist of parameters they will accept or a
* blacklist of paramters they will reject to prevent clients from setting other unexpected (and possibly dangerous)
* parameters.
*
* Using {@link ParameterNameAware} could be dangerous as {@link ParameterNameAware#acceptableParameterName(String)} takes precedence
* over {@link ParametersInterceptor} which means if ParametersInterceptor excluded given parameter name you can accept it with
* {@link ParameterNameAware#acceptableParameterName(String)}.
*
* The best idea is to define very tight restrictions with ParametersInterceptor and relax them per action with
* {@link ParameterNameAware#acceptableParameterName(String)}
*
* <!-- END SNIPPET: javadoc -->
*
Expand Down

0 comments on commit 49d339f

Please sign in to comment.