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

Sergey Beryozkin commented on CXF-4281:
---------------------------------------

Hi, I think 

> UriBuilder.fromPath(null) doesn't throw any exception. It should throw a 
> IllegalArgumentException, no?

Yes - which actually highlights the problem with the API implementation that 
you use, recall you mentioning
URIBuilder.fromPath() calls replacePath() [1] on the newly created UriBuilder, 
where supplying 'null' is perfectly valid.
Effectively we are talking about the API in question breaking the 'negative' 
part of the UriBuilder.fromPath contract



[1] 
http://docs.oracle.com/javaee/6/api/javax/ws/rs/core/UriBuilder.html#replacePath%28java.lang.String%29
                
> uribuilder from null?
> ---------------------
>
>                 Key: CXF-4281
>                 URL: https://issues.apache.org/jira/browse/CXF-4281
>             Project: CXF
>          Issue Type: Bug
>            Reporter: Romain Manni-Bucau
>
> in tomee/openejb we need to patch the uri builder to be able to satisfied 
> some cases (UriBuilder.fromPath(null) for instance)
> here is the patched builder working for us: 
> http://svn.apache.org/repos/asf/openejb/trunk/openejb/server/openejb-cxf-rs/src/main/java/org/apache/openejb/server/cxf/rs/OpenEJBRuntimeDelegateImpl.java
> any way to get it merged with cxf?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to