Merge pull request #24 from yehuda-elementryx/master

brandonmwest · web-flow · commit e2bde086bc40 · 2020-04-10T10:05:04.000-07:00
Fix bug in issue #23. Fix check ensuring socket is open before sendin…
diff --git a/lib/main.js b/lib/main.js
@@ -67,7 +67,7 @@ let streamAudioToWebSocket = function (userMediaStream) {
             // the audio stream is raw audio bytes. Transcribe expects PCM with additional metadata, encoded as binary
             let binary = convertAudioToBinaryMessage(rawAudioChunk);
 
-            if (socket.OPEN)
+            if (socket.readyState === socket.OPEN)
                 socket.send(binary);
         }
     )};
@@ -148,7 +148,7 @@ let handleEventStreamMessage = function (messageJson) {
 }
 
 let closeSocket = function () {
-    if (socket.OPEN) {
+    if (socket.readyState === socket.OPEN) {
         micStream.stop();
 
         // Send an empty frame so that Transcribe initiates a closure of the WebSocket after submitting all transcripts

Original file line number	Diff line number	Diff line change
`@@ -67,7 +67,7 @@ let streamAudioToWebSocket = function (userMediaStream) {`
`67`	`67`	`// the audio stream is raw audio bytes. Transcribe expects PCM with additional metadata, encoded as binary`
`68`	`68`	`let binary = convertAudioToBinaryMessage(rawAudioChunk);`
`69`	`69`
`70`		`- if (socket.OPEN)`
	`70`	`+ if (socket.readyState === socket.OPEN)`
`71`	`71`	`socket.send(binary);`
`72`	`72`	`}`
`73`	`73`	`)};`
`@@ -148,7 +148,7 @@ let handleEventStreamMessage = function (messageJson) {`
`148`	`148`	`}`
`149`	`149`
`150`	`150`	`let closeSocket = function () {`
`151`		`- if (socket.OPEN) {`
	`151`	`+ if (socket.readyState === socket.OPEN) {`
`152`	`152`	`micStream.stop();`
`153`	`153`
`154`	`154`	`// Send an empty frame so that Transcribe initiates a closure of the WebSocket after submitting all transcripts`