activemq-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Christian Kemper (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (AMQ-5865) Enable "getRemoteAddress()" method in WebSocket Requests
Date Fri, 26 Jun 2015 05:26:04 GMT

     [ https://issues.apache.org/jira/browse/AMQ-5865?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Christian Kemper updated AMQ-5865:
----------------------------------

see also http://activemq.2283324.n4.nabble.com/Enable-quot-getRemoteAddress-quot-method-in-WebSocket-Requests-tt4698179.html

> Enable "getRemoteAddress()" method in WebSocket Requests
> --------------------------------------------------------
>
>                 Key: AMQ-5865
>                 URL: https://issues.apache.org/jira/browse/AMQ-5865
>             Project: ActiveMQ
>          Issue Type: Bug
>          Components: MQTT, stomp
>    Affects Versions: 5.11.0, 5.11.1
>         Environment: all
>            Reporter: Christian Kemper
>            Priority: Trivial
>              Labels: easyfix, easytest, features, javascript, patch
>             Fix For: 5.11.2, 5.12.0, 6.0.0
>
>         Attachments: CustomStompSocket.java, CustomWebSocketServlet.java
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> The method (getRemoteAddress()) in class "org.apache.activemq.transport.ws.AbstractStompSocket"
does not return the correct address; it returns always ""StompSocket_" + hashCode();" 
> We need the correct address (e.g. "wss://127.0.0.1:63114") for our use case. 
> The implementation whould be very simple. 
> In class "org.apache.activemq.transport.ws.jetty8.WSServlet" (and also in jetty9 package)
pass the address during creation of class StompSocket in the same package: 
> String scheme = request.getScheme();
> if ("http".equalsIgnoreCase(scheme)) {
> 	scheme = "ws";
> } else if ("https".equalsIgnoreCase(scheme)) {
> 	scheme = "wss";
> 		}
> WebSocket socket = new CustomStompSocket(scheme + "://" + request.getRemoteAddr() + ":"
+ request.getRemotePort());
> It meight be that MQTT is also affected; a fix should be done by the same way.



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

Mime
View raw message