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

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

Lets see how CXF 2.5.x build reacts to 
http://svn.apache.org/viewvc?rev=1333860&view=rev :-)

As for the 
'UriBuilder.fromUri("http://foo:1234";).scheme(null).build().getRawSchemeSpecificPart()
 != null', I think that setting the scheme to 'null' should only affect the 
'http' part from the original URI which would leave the scheme specific part as 
'foo:1234'. 

Write now I can see CXF printing "//localhost:8080" in this case which is 
useless. I think the right outcome should be either "http://foo:1234"; where 
UriBuilder detects that scheme specific part is set but the actual scheme is 
not and thus defaults to 'http' or throw IllegalArgumentException at 
scheme(null)

                
> 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