-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/7602/
-----------------------------------------------------------

(Updated Oct. 17, 2012, 12:40 a.m.)


Review request for wave, Michael MacFadden, Yuri Zelikov, and Ali Lown.


Changes
-------

New patch following Ali comments.


Description
-------

This patch fix WAVE-376, that is, detect that websocket it's not working and 
try to use socket.io instead.

This patch depends on https://reviews.apache.org/r/7600/ to work properly.


This addresses bug WAVE-376.
    https://issues.apache.org/jira/browse/WAVE-376


Diffs (updated)
-----

  src/org/waveprotocol/box/webclient/client/WaveWebSocketClient.java 5645f5e 
  src/org/waveprotocol/box/webclient/client/WebClient.java 863ae6c 

Diff: https://reviews.apache.org/r/7602/diff/


Testing
-------

How to test this:
- Configure your local WIAB under a http_proxy
- or:
1) Configure your server websocket with a different name for websocket address, 
for instance ws.localhost
2) configure it in the /etc/hosts
127.0.0.1 ws.localhost
3) run the server
4) test that all works ok using websocket
5) remove the entry from /etc/hosts
6) refresh your browser and check that websocket is not working and the client 
use socketio instead.


Thanks,

Vicente J. Ruiz Jurado

Reply via email to