ssh-client-wrapper 2.2.0 → 2.2.1

package/lib/fork.js

@@ -58,7 +58,7 @@ function sshLoginCallback(output, ptyProcess, pw, ph, logger = debugSendPty) {
 /**
  * create pty for master ssh session and store to hostInfo.masterPty
  */
-function createMastePty(hostInfo) {
+function createMasterPty(hostInfo) {
   debug("fork pty for master connection");
 
   if (hostInfo.masterPty) {
@@ -125,6 +125,6 @@ async function fork(hostInfo, cmd, args, timeout, outputCallback) {
 }
 module.exports = {
   fork,
-  createMastePty,
+  createMasterPty,
   sshLoginCallback
 };

package/lib/sshExec.js

@@ -5,7 +5,7 @@ const debug = require("debug")("sshClientWrapper:debug:sshExec");
 const debugVerbose = require("debug")("sshClientWrapper:verbose:sshExec");
 const debugSendMasterPty = require("debug")("sshClientWrapper:insecure:sshExec_masterPty");
 const { sendPty, sshCmd, getSshOption, sanityCheck, watchDogTimer } = require("./util.js");
-const { fork, createMastePty, sshLoginCallback } = require("./fork.js");
+const { fork, createMasterPty, sshLoginCallback } = require("./fork.js");
 
 /**
  * Execute command on remote host.
@@ -15,6 +15,7 @@ const { fork, createMastePty, sshLoginCallback } = require("./fork.js");
  * @param {Function} outputCallback - Callback routine for stdout and stderr.
  */
 async function sshExec(hostInfo, cmd, timeout, outputCallback) {
+  debug("sshExec called");
   await connect(hostInfo);
   debug("exec", cmd, "on remote server");
   const args = getSshOption(hostInfo, false);
@@ -46,16 +47,13 @@ async function sshExec(hostInfo, cmd, timeout, outputCallback) {
  * @param {Integer} timeout - timeout in secconds
  */
 async function canConnect(hostInfo, timeout = 60) {
-  debug("sanity check");
+  debug("canConnect called");
   sanityCheck(hostInfo);
 
-  debug("check connectibity");
-
-  try {
-    await connect(hostInfo, timeout);
-  } finally {
-    await disconnect(hostInfo);
-  }
+  await connect(hostInfo, timeout);
+  await disconnect(hostInfo);
+  //if connection fail, Error is throwed.
+  //so, if we reach here, connection must be succeeded
   return true;
 }
 
@@ -97,16 +95,17 @@ async function existsMaster(hostInfo, argTimeout) {
  * @param {Object} hostInfo - hostinfo object
  */
 async function connect(hostInfo, timeout = 60) {
+  debug("connect called");
+
   if (await existsMaster(hostInfo, timeout)) {
     debugVerbose("master connection exists");
     return;
   }
 
   debug(`create master pty for ${hostInfo.host}`);
-  createMastePty(hostInfo, debugSendMasterPty);
+  createMasterPty(hostInfo, debugSendMasterPty);
 
   debug(`create master ssh session to ${hostInfo.host}`);
-
   const words = crypto.randomUUID();
   const re = new RegExp(`^${words}`);
 
@@ -177,6 +176,8 @@ async function connect(hostInfo, timeout = 60) {
  * @param {Object} hostInfo - hostinfo object
  */
 async function disconnect(hostInfo) {
+  debug("disconnect called");
+
   if (await existsMaster(hostInfo)) {
     const args = getSshOption(hostInfo);
     args.push("-Oexit");
@@ -218,6 +219,7 @@ async function disconnect(hostInfo) {
  * @returns {string[]} - output from ls
  */
 async function ls(hostInfo, target, lsOpt, timeout) {
+  debug("ls called");
   const outputArray = [];
   try {
     await sshExec(hostInfo, `ls ${lsOpt.join(" ")} ${target}`, timeout, (data)=>{

package/lib/util.js

@@ -26,6 +26,8 @@ const buff2String = (func, data)=>{
  * @param { Object} hostInfo - host information object defined in index.js
  */
 const sanityCheck = (hostInfo)=>{
+  debug("sanityCheck called");
+
   if (typeof hostInfo.host !== "string" || hostInfo.host === "") {
     throw new Error("host must be non-empty string");
   }
@@ -43,6 +45,7 @@ const sanityCheck = (hostInfo)=>{
  * @param {boolean} withoutDestination - return except for hostname
  */
 const getSshOption = (hostInfo, withoutDestination)=>{
+  debug("getSshOption called");
   const args = [];
 
   if (!withoutDestination) {
@@ -140,7 +143,7 @@ const watchDogTimer = (timeout, label, cb)=>{
     })
     .catch((err)=>{
       if (err.name === "AbortError") {
-        debug(`timeout was aborted because ${label} has done`);
+        debug(`timeout was aborted because ${label} has finished or canceled`);
       } else {
         throw err;
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ssh-client-wrapper",
-  "version": "2.2.0",
+  "version": "2.2.1",
   "description": "wrapper library for open ssh client and rsync",
   "keywords": [
     "ssh",