[jboss-jira] [JBoss JIRA] Commented: (JBREM-569) HTTP(S) proxy broken

[Tom Elrod (JIRA)]

    [ http://jira.jboss.com/jira/browse/JBREM-569?page=comments#action_12340854 ] 
            
Tom  Elrod commented on JBREM-569:
----------------------------------

One additional note:

You seem to use the originalURI in the transport() method of HTTPClientInvoker class:
String targetURL = getLocator().getOriginalURI();

But i think that getLocator().getURI() is more correct.
For example, when using annotations we sometimes use the following:
@RemoteBinding(clientBindUrl="http://0.0.0.0:8888/SharedStateManager",jndiBinding="SharedStateManagerhttp")

0.0.0.0 is substituted by the real IP-number of the server during the deployment.
Therefore, the originalURI of the locatoris http://0.0.0.0:8888/SharedStateManager,
but URI field is correct and set to: http://x.y.z.w:8888/SharedStateManager
where x.y.z.w is the IP of the server.

With your current code, there is no easy way for a client to get the correct locator, since http://0.0.0.0:8888/SharedStateManager is returned. But may be it is also the same bug somewhere else, which makes use of originalURI, instead of URI.

The fix is obvious and we hope to see it soon in the main branch.

Best regards,
 Ivan

> HTTP(S) proxy broken
> --------------------
>
>                 Key: JBREM-569
>                 URL: http://jira.jboss.com/jira/browse/JBREM-569
>             Project: JBoss Remoting
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>          Components: transport
>    Affects Versions: 2.0.0.CR1 (Boon)
>            Reporter: Tom  Elrod
>         Assigned To: Tom  Elrod
>             Fix For: 2.0.0.GA (Boon)
>
>
> Per comments to JBREM-27, http proxy seems to be broken.

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