struts.allowed.action.names inconsistency

I think there is a inconsistency in DefaultActionMapper.cleanupActionName

According to http://struts.apache.org/development/2.x/docs/s2-015.html struts.allowed.action.names defines a regex the action name must match.

The default regex is:

[a-z]*[A-Z]*[0-9]*[.\-_!/]*

I have an action with the name "core_blz"

This action name does not match the regex (underscore is not at the end)(maybe the default value should be changed).

DefaultActionMapper reports Action [#0] do not match allowed action names pattern [#1], cleaning it up!

But the cleaned action name is still core_blz.

If this function is only to remove suspicious characters, then the warning should not be displayed.