andreacavalli/netty5
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix incoherence in WebSocket example

Browse Source 
Motivation:

The logic in the current websocket example is confusing and misleading

Modifications:

Remove occurrences of "http" and "https" and replace them with "ws" and "wss"

Result:

The example code is now coherent and is easier to understand for a new user.
This commit is contained in:
Jean-Rémi Desjardins 2015-06-01 17:08:55 -07:00 committed by Norman Maurer
parent c999dc6f20
commit e19e1e29d8
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
example/src/main/java/io/netty/example/http/websocketx/client/WebSocketClient.java
View File

@ -62,13 +62,13 @@ public final class WebSocketClient {
public static void main(String[] args) throws Exception { public static void main(String[] args) throws Exception {
URI uri = new URI(URL); URI uri = new URI(URL);
String scheme = uri.getScheme() == null? "http" : uri.getScheme(); String scheme = uri.getScheme() == null? "ws" : uri.getScheme();
final String host = uri.getHost() == null? "127.0.0.1" : uri.getHost(); final String host = uri.getHost() == null? "127.0.0.1" : uri.getHost();
final int port; final int port;
if (uri.getPort() == -1) { if (uri.getPort() == -1) {
if ("http".equalsIgnoreCase(scheme)) { if ("ws".equalsIgnoreCase(scheme)) {
port = 80; port = 80;
} else if ("https".equalsIgnoreCase(scheme)) { } else if ("wss".equalsIgnoreCase(scheme)) {
port = 443; port = 443;
} else { } else {
port = -1; port = -1;