You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openwhisk.apache.org by dg...@apache.org on 2023/05/03 14:27:57 UTC

[openwhisk-runtime-nodejs] branch master updated: Make error loging clearer (#238)

This is an automated email from the ASF dual-hosted git repository.

dgrove pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/openwhisk-runtime-nodejs.git


The following commit(s) were added to refs/heads/master by this push:
     new 577ccbb  Make error loging clearer (#238)
577ccbb is described below

commit 577ccbb34ecaf9ffa2b836640432669e53c3314a
Author: Luke-Roy-IBM <83...@users.noreply.github.com>
AuthorDate: Wed May 3 16:27:51 2023 +0200

    Make error loging clearer (#238)
    
    * Update error logs
---
 core/nodejsActionBase/runner.js | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/core/nodejsActionBase/runner.js b/core/nodejsActionBase/runner.js
index 5ed211c..84ae1c7 100644
--- a/core/nodejsActionBase/runner.js
+++ b/core/nodejsActionBase/runner.js
@@ -29,7 +29,7 @@ function initializeActionHandler(message) {
         // The code is a base64-encoded zip file.
         ext = detectFileType(message.code)
         if (ext == 'unsupported'){
-            return Promise.reject("There was an error uncompressing the action archive.");
+            return Promise.reject("There was an error uncompressing the action archive. The file type is unsupported");
         }
         return extractInTmpDir(message.code)
             .then(moduleDir => {
@@ -161,13 +161,13 @@ function extractInTmpDir(archiveFileContents) {
             if (ext === 'zip') {
                 return exec("unzip -qq " + archiveFile + " -d " + tmpDir)
                 .then(res => path.resolve(tmpDir))
-                .catch(error => Promise.reject("There was an error uncompressing the action archive."));
+                .catch(error => Promise.reject("There was an error uncompressing the action Zip archive."));
             } else if (ext === 'tar.gz') {
                 return exec("tar -xzf " + archiveFile + " -C " + tmpDir + " > /dev/null")
                 .then(res => path.resolve(tmpDir))
-                .catch(error => Promise.reject("There was an error uncompressing the action archive."));
+                .catch(error => Promise.reject("There was an error uncompressing the action Tar GZ archive."));
             } else {
-                  return Promise.reject("There was an error uncompressing the action archive.");
+                  return Promise.reject("There was an error uncompressing the action archive. file ext did not Match");
             }
         });
     });