struts-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Aleksandr Mashchenko (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (WW-4585) Struts2 Rest plugin doesn't handle JSESSIONID with DMI
Date Fri, 15 Jan 2016 08:06:39 GMT

    [ https://issues.apache.org/jira/browse/WW-4585?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15101413#comment-15101413
] 

Aleksandr Mashchenko commented on WW-4585:
------------------------------------------

[~rpii] Yes, open a new issue and explain in it what do you expect to get. Right now test
results show `Expected :dog` but there is no such assert in the test method you've provided.
If you have a solution base it on master branch, please. If you need this in 2.3.x branch
then we can cherry pick it from master. Additionally, instead of providing patches you can
open a Pull Request in the [GitHub|https://github.com/apache/struts].

> 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)

Mime
View raw message