[jira] [Commented] (CXF-6785) AbstractHTTPDestination should handle the case when pathInfo of HttpServletRequest is null

[Sergey Beryozkin (JIRA)]

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

Sergey Beryozkin commented on CXF-6785:
---------------------------------------

Hi Freeman

Thanks, I understand some HTTP stacks can return null and indeed it is 
documented in HttpServletRequest that returning null is possible, that is fine.

But what does 'requestUri' return if pathInfo is null ?
As I said, are you sure 'contextServletPath' is not lost, i.e, should we have:
{code:java}
if (pathInfo != null) {
        inMessage.put(Message.PATH_INFO, contextServletPath + pathInfo);
} else {
       inMessage.put(Message.PATH_INFO, contextServletPath); 
       // instead of
       // inMessage.put(Message.PATH_INFO, requestURI);
}
{code}

Cheers, Sergey


> AbstractHTTPDestination should handle the case when pathInfo of 
> HttpServletRequest is null 
> -------------------------------------------------------------------------------------------
>
>                 Key: CXF-6785
>                 URL: https://issues.apache.org/jira/browse/CXF-6785
>             Project: CXF
>          Issue Type: Bug
>            Reporter: Freeman Fang
>            Assignee: Freeman Fang
>             Fix For: 3.2.0, 3.1.6
>
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)