[jboss-jira] [JBoss JIRA] Closed: (JBREM-1175) URL-component-based InvokerLocator constructor generates incorrect locator string under IPv6

[Ron Sigal (JIRA)]

     [ https://jira.jboss.org/jira/browse/JBREM-1175?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ron Sigal closed JBREM-1175.
----------------------------

    Resolution: Done


Tests passing in hudson.

> URL-component-based InvokerLocator constructor generates incorrect locator string under IPv6 
> ---------------------------------------------------------------------------------------------
>
>                 Key: JBREM-1175
>                 URL: https://jira.jboss.org/jira/browse/JBREM-1175
>             Project: JBoss Remoting
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>          Components: unifiedinvoker
>    Affects Versions: 2.2.3.SP1, 2.5.2.SP1 (Flounder)
>            Reporter: Richard Achmatowicz
>            Assignee: Ron Sigal
>             Fix For: 2.5.2.SP2 (Flounder), 2.2.3.SP2
>
>
> The constructor    
> public InvokerLocator(String protocol, String host, int port, String path, Map parameters)
> creates a uri and originalURI from the component parts of the URL passed into it. 
> This uri is incorrectly formed when the host is an IPv6 IP address literal, as square brackets need to be added to IPv6 URLs.
> NOTE: the other constructor
> public InvokerLocator(String uri)
> does work correctly in the non-lagacy parsing case as the method URIParse will parse a valid IPv6 URL into component parts using the URI constructor. This constructor parses an input URI string into component parts in such a way that the hostname will retain its square brackets if the URI represents a well formed IPv6 URL - and these square brackets do not need to be added when the string uri is reconstructed later on oin the method.
>  

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira