Merge pull request #123 from rrayst/master

brendandburns · web-flow · commit fc13f8f1f54f · 2017-11-22T20:59:48.000-08:00
fixed WebSockets message types: binary vs text
diff --git a/util/src/main/java/io/kubernetes/client/util/WebSockets.java b/util/src/main/java/io/kubernetes/client/util/WebSockets.java
@@ -125,9 +125,9 @@ public void onOpen(final WebSocket webSocket, Response response) {
         @Override
         public void onMessage(ResponseBody body) throws IOException {
             if (body.contentType() == TEXT) {
-                listener.bytesMessage(body.byteStream());
-            } else if (body.contentType() == BINARY) {
                 listener.textMessage(body.charStream());
+            } else if (body.contentType() == BINARY) {
+                listener.bytesMessage(body.byteStream());
             }
             body.close();
         }

Original file line number	Diff line number	Diff line change
`@@ -125,9 +125,9 @@ public void onOpen(final WebSocket webSocket, Response response) {`
`125`	`125`	`@Override`
`126`	`126`	`public void onMessage(ResponseBody body) throws IOException {`
`127`	`127`	`if (body.contentType() == TEXT) {`
`128`		`- listener.bytesMessage(body.byteStream());`
`129`		`- } else if (body.contentType() == BINARY) {`
`130`	`128`	`listener.textMessage(body.charStream());`
	`129`	`+ } else if (body.contentType() == BINARY) {`
	`130`	`+ listener.bytesMessage(body.byteStream());`
`131`	`131`	`}`
`132`	`132`	`body.close();`
`133`	`133`	`}`