Merge pull request #4286 from NativeScript/vladimirov/fix-index-out-of-range-debug

fix: errors during debug on iOS

Author: rosen-vladimirov

lib/device-sockets/ios/app-debug-socket-proxy-factory.ts

@@ -1,9 +1,9 @@
 import { EventEmitter } from "events";
 import { CONNECTION_ERROR_EVENT_NAME } from "../../constants";
-import { PacketStream } from "./packet-stream";
 import * as net from "net";
 import * as ws from "ws";
 import temp = require("temp");
+import { MessageUnpackStream } from "ios-device-lib";
 
 export class AppDebugSocketProxyFactory extends EventEmitter implements IAppDebugSocketProxyFactory {
 	private deviceWebServers: IDictionary<ws.Server> = {};
@@ -134,11 +134,16 @@ export class AppDebugSocketProxyFactory extends EventEmitter implements IAppDebu
 			const encoding = "utf16le";
 
 			const appDebugSocket: net.Socket = (<any>req)["__deviceSocket"];
-			const packets = new PacketStream();
+			const packets = new MessageUnpackStream();
 			appDebugSocket.pipe(packets);
 
 			packets.on("data", (buffer: Buffer) => {
-				webSocket.send(buffer.toString(encoding));
+				const message = buffer.toString(encoding);
+				if (webSocket.readyState === webSocket.OPEN) {
+					webSocket.send(message);
+				} else {
+					this.$logger.trace(`Received message ${message}, but unable to send it to webSocket as its state is: ${webSocket.readyState}`);
+				}
 			});
 
 			webSocket.on("error", err => {

lib/device-sockets/ios/packet-stream.ts

@@ -45,7 +45,7 @@
     "gaze": "1.1.0",
     "iconv-lite": "0.4.11",
     "inquirer": "6.2.0",
-    "ios-device-lib": "0.4.15",
+    "ios-device-lib": "0.5.0",
     "ios-mobileprovision-finder": "1.0.10",
     "ios-sim-portable": "4.0.6",
     "istextorbinary": "2.2.1",