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 ed3d26cf7d
commit 9bcfef0f10
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

@ -61,13 +61,13 @@ public final class WebSocketClient {
public static void main(String[] args) throws Exception {
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 int port;
if (uri.getPort() == -1) {
if ("http".equalsIgnoreCase(scheme)) {
if ("ws".equalsIgnoreCase(scheme)) {
port = 80;
} else if ("https".equalsIgnoreCase(scheme)) {
} else if ("wss".equalsIgnoreCase(scheme)) {
port = 443;
} else {
port = -1;