[CopyFilesOverSSH] Addressed code review points

microsoft · Jul 21, 2020 · 9298ae8 · 9298ae8
1 parent b9a6cfd
commit 9298ae8
Show file tree

Hide file tree

Showing 3 changed files with 87 additions and 109 deletions.
diff --git a/Tasks/CopyFilesOverSSHV0/copyfilesoverssh.ts b/Tasks/CopyFilesOverSSHV0/copyfilesoverssh.ts
@@ -222,7 +222,7 @@ async function run() {
                     let targetPath = path.posix.join(targetFolder, relativePath);
 
                     if (!path.isAbsolute(targetPath)) {
-                        targetPath = './' + targetPath
+                        targetPath = `./${targetPath}`;
                     }
 
                     console.log(tl.loc('StartedFileCopy', fileToCopy, targetPath));

diff --git a/Tasks/CopyFilesOverSSHV0/package-lock.json b/Tasks/CopyFilesOverSSHV0/package-lock.json
diff --git a/Tasks/CopyFilesOverSSHV0/sshhelper.ts b/Tasks/CopyFilesOverSSHV0/sshhelper.ts
@@ -53,6 +53,13 @@ export class SshHelper {
         await defer.promise;
     }
 
+    private unixyPath(filepath) {
+        if (process.platform === 'win32') {
+            return filepath.replace(/\\/g, '/');
+        }
+        return filepath;
+    }
+
     /**
      * Sets up the SSH connection
      */
@@ -98,9 +105,7 @@ export class SshHelper {
      * @returns {Promise<string>}
      */
     async uploadFile(sourceFile: string, dest: string) : Promise<string> {
-        if (process.platform === 'win32') {
-            dest = dest.replace(/\\/g, '/');
-        }
+        dest = this.unixyPath(dest);
 
         tl.debug('Upload ' + sourceFile + ' to ' + dest + ' on remote machine.');
 
@@ -142,7 +147,7 @@ export class SshHelper {
         if(!this.sftpClient) {
             defer.reject(tl.loc('ConnectionNotSetup'));
         }
-        if (await this.sftpClient.stat(path)) {
+        if (await this.sftpClient.exsts(path)) {
             //path exists
             defer.resolve(true);
         } else {