Instead of directly throwing error, UAC wrapper emits error event

when remote reports error message
JLChnToZ · Dec 31, 2018 · 73895e2 · 73895e2
1 parent e952bc3
commit 73895e2
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 5 deletions.
diff --git a/lib/terminals/uacwrapper.js b/lib/terminals/uacwrapper.js
diff --git a/lib/terminals/uacwrapper.js.map b/lib/terminals/uacwrapper.js.map
diff --git a/src/terminals/uacwrapper.ts b/src/terminals/uacwrapper.ts
@@ -37,7 +37,7 @@ export class UACClient extends TerminalBase<Socket> {
     // Create a named pipe for IPC between clones.
     const pipe = `uniterm-${process.pid}-${(await randomBytesAsync(16)).toString('hex')}`;
     createServer()
-    .listen(joinPath(`\\\\.\\pipe`, pipe))
+    .listen(joinPath('\\\\.\\pipe', pipe))
     .once('connection', this.handleConnection)
     .once('connection', closeServer);
     this._pushData('SUDO: Waiting to confirm getting administrator privileges...\r\n');
@@ -148,7 +148,8 @@ export class UACClient extends TerminalBase<Socket> {
           case CMDType.Error:
             dataSize += buffer.readUInt32BE(1) + 4;
             if(buffer.length < dataSize) return;
-            throw new Error(`Remote error: ${buffer.toString('utf8', 5, dataSize)}`);
+            this.emit('error', new Error(`Remote error: ${buffer.toString('utf8', 5, dataSize)}`));
+            break;
           case CMDType.Exit:
             dataSize += 4;
             if(buffer.length < dataSize) return;