fix: log status message only once and without prefix (#179)

apify · Mar 23, 2023 · 1f11a6a · 1f11a6a
1 parent 27edd99
commit 1f11a6a
Showing 1 changed file with 6 additions and 8 deletions.
diff --git a/packages/apify/src/actor.ts b/packages/apify/src/actor.ts
@@ -250,13 +250,11 @@ export class Actor<Data extends Dictionary = Dictionary> {
 
         if (options.exitCode > 0) {
             options.statusMessage ??= `Actor finished with an error (exit code ${options.exitCode})`;
-            log.error(options.statusMessage);
         } else {
             options.statusMessage ??= `Actor finished successfully (exit code ${options.exitCode})`;
-            log.info(options.statusMessage);
         }
 
-        await this.setStatusMessage(options.statusMessage, { isStatusMessageTerminal: true });
+        await this.setStatusMessage(options.statusMessage, { isStatusMessageTerminal: true, level: options.exitCode > 0 ? LogLevel.ERROR : LogLevel.INFO });
 
         if (!options.exit) {
             return;
@@ -519,20 +517,20 @@ export class Actor<Data extends Dictionary = Dictionary> {
         ow(statusMessage, ow.string);
         ow(isStatusMessageTerminal, ow.optional.boolean);
 
-        statusMessage = `[Status message]: ${statusMessage}`;
+        const loggedStatusMessage = `[Status message]: ${statusMessage}`;
 
         switch (level) {
             case LogLevel.DEBUG:
-                log.debug(statusMessage);
+                log.debug(loggedStatusMessage);
                 break;
             case LogLevel.WARNING:
-                log.warning(statusMessage);
+                log.warning(loggedStatusMessage);
                 break;
             case LogLevel.ERROR:
-                log.error(statusMessage);
+                log.error(loggedStatusMessage);
                 break;
             default:
-                log.info(statusMessage);
+                log.info(loggedStatusMessage);
                 break;
         }