[
https://issues.apache.org/jira/browse/WW-5024?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16806364#comment-16806364
]
ASF GitHub Bot commented on WW-5024:
------------------------------------
JCgH4164838Gh792C124B5 commented on pull request #346: Forward port fix for
WW-5024 to 2.6:
URL: https://github.com/apache/struts/pull/346
Forward port fix for WW-5024 to 2.6:
- **NOTE**: _If the PR is accepted_ **please credit Robert Hollencamp
(Github @rhollencamp)** for this change
as he found the issue, proposed a solution for 2.6 (master), and
opened the JIRA.
- Updated HttpParameters, ActionMappingParametersInterceptor to prevent
multi-level Parameter wrapping from occuring.
- Updated HttpParameters create(), build() and buildNoNestedWrapping() to
replace redundant type operators with <> based on IDE hint.
- Made HttpParameters parameters member final based on IDE hint.
- Added a new ActionMappingParametersInterceptorTest to verify the fix.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
> HttpParameters.Builder can wrap objects in two layers of Parameters
> -------------------------------------------------------------------
>
> Key: WW-5024
> URL: https://issues.apache.org/jira/browse/WW-5024
> Project: Struts 2
> Issue Type: Bug
> Components: Value Stack
> Affects Versions: 2.5.20
> Reporter: Robert Hollencamp
> Priority: Major
>
> Parameters added to ActionMapping can end up wrapped in Parameter twice
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
