[
https://issues.apache.org/jira/browse/WW-4585?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15098567#comment-15098567
]
Aleksandr Mashchenko edited comment on WW-4585 at 1/14/16 7:03 PM:
-------------------------------------------------------------------
[~rpii] Fixed it in master branch also which has newer version than 2.3.24.1,
that's why fix is different from yours. Next time try to preserve original
source formatting when submitting patches/PR-s. Thank you for reporting. :)
was (Author: aleksandr-m):
[~rpii] Fixed it in master branch also which has newer version than 2.3.34.1,
that's why fix is different from yours. Next time try to preserve original
source formatting when submitting patches/PR-s. Thank you for reporting. :)
> Struts2 Rest plugin doesn't handle JSESSIONID with DMI
> ------------------------------------------------------
>
> Key: WW-4585
> URL: https://issues.apache.org/jira/browse/WW-4585
> Project: Struts 2
> Issue Type: Bug
> Components: Plugin - REST
> Affects Versions: 2.3.24
> Environment: tomcat
> Reporter: Rich P Peters, II
> Assignee: Aleksandr Mashchenko
> Priority: Blocker
> Fix For: 2.3.25, 2.5
>
> Attachments: RestActionMapper.java
>
>
> If a url with DMI and with a JSESSIONID is sent to a Struts2 action, a
> failure will result
> see this modified unit test from the plugin unit test:
> {code:java}
> public void testGetJsessionIdSemicolonMappingWithMethod() throws
> Exception {
>
> req.setRequestURI("/myapp/animals/dog/fido!update;jsessionid=29fefpv23do1g");
> req.setServletPath("/animals/dog/fido");
> req.setMethod("GET");
> mapper.setAllowDynamicMethodCalls("true");
> ActionMapping mapping = mapper.getMapping(req, configManager);
> assertEquals("/animals", mapping.getNamespace());
> assertEquals("dog", mapping.getName());
> assertEquals("fido", ((String[]) mapping.getParams().get("id"))[0]);
> assertEquals("update", mapping.getMethod());
> }
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
