----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7602/#review12638 -----------------------------------------------------------
Ship it! Ship It! - Yuri Zelikov On Oct. 17, 2012, 12:40 a.m., Vicente J. Ruiz Jurado wrote: > > ----------------------------------------------------------- > 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. > > > 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 > ----- > > 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 > >
