feat: allow failure file env-var to prevent logging (#4982)

addresses #3832
googleapis · Jan 27, 2022 · eacb8a1 · eacb8a1
1 parent 48cf3a9
commit eacb8a1
Showing 1 changed file with 11 additions and 3 deletions.
diff --git a/Core/src/Batch/HandleFailureTrait.php b/Core/src/Batch/HandleFailureTrait.php
@@ -43,6 +43,12 @@ trait HandleFailureTrait
     private function initFailureFile()
     {
         $this->baseDir = getenv('GOOGLE_CLOUD_BATCH_DAEMON_FAILURE_DIR');
+
+        if ('false' === $this->baseDir) {
+            // setting the file to the string "false" will prevent logging of failed items
+            return;
+        }
+
         if ($this->baseDir === false) {
             $this->baseDir = sprintf(
                 '%s/batch-daemon-failure',
@@ -80,9 +86,11 @@ public function handleFailure($idNum, array $items)
             $this->initFailureFile();
         }
 
-        $fp = @fopen($this->failureFile, 'a');
-        @fwrite($fp, serialize([$idNum => $items]) . PHP_EOL);
-        @fclose($fp);
+        if ($this->failureFile) {
+            $fp = @fopen($this->failureFile, 'a');
+            @fwrite($fp, serialize([$idNum => $items]) . PHP_EOL);
+            @fclose($fp);
+        }
     }
 
     /**