struts-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Lukasz Lenart (JIRA)" <>
Subject [jira] [Commented] (WW-4171) getText methods are not documented as evaluating OGNL
Date Tue, 06 Aug 2013 06:24:49 GMT


Lukasz Lenart commented on WW-4171:

I think you a bit demonise here. The first parameter isn't evaluated as an OGNL expression
- is just a key used to look up in a resource bundle:

{code:java|title=LocalizedTextUtil, line 683}
String message = TextParseUtil.translateVariables(bundle.getString(key), valueStack);

and just the value from resource bundle is evaluated as an expression which is rather obvious
if you have something like this in a properties file

requiredstring = ${getText(fieldName)} is required.

I have added a note about evaluation to the docs
> getText methods are not documented as evaluating OGNL
> -----------------------------------------------------
>                 Key: WW-4171
>                 URL:
>             Project: Struts 2
>          Issue Type: Bug
>          Components: Documentation
>    Affects Versions:
>            Reporter: Coverity Security Research Laboratory
>              Labels: security
> The methods below evaluate OGNL as their first parameter. However they are not documented
as evaluating OGNL. We have observed this occurring in one project and are contacting the
affected vendors.
> com.opensymphony.xwork2.TextProviderSupport.getText(String, String[])
> com.opensymphony.xwork2.TextProviderSupport.getText(String, List<?>)
> com.opensymphony.xwork2.TextProviderSupport.getText(String)
> These methods are then used by ActionSupport (via its getText methods). None of these
methods are documented as evaluating OGNL either.
> This issue is recommending that all of these methods are documented as evaluating OGNL
since this may come as a surprise to some developers.

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see:

View raw message