Skip to content

Commit cabd25a

Browse files
authored
Merge pull request #8864 from ProcessMaker/task/FOUR-31810
FOUR-31810 Only output stack traces for debug logs
2 parents cf325b3 + 7f6635b commit cabd25a

2 files changed

Lines changed: 6 additions & 5 deletions

File tree

ProcessMaker/Jobs/RunServiceTask.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,9 @@ class RunServiceTask extends BpmnAction implements ShouldQueue
3636
/**
3737
* Create a new job instance.
3838
*
39-
* @param \ProcessMaker\Models\Process $definitions
40-
* @param \ProcessMaker\Models\ProcessRequest $instance
41-
* @param \ProcessMaker\Models\ProcessRequestToken $token
39+
* @param Definitions $definitions
40+
* @param ProcessRequest $instance
41+
* @param ProcessRequestToken $token
4242
* @param array $data
4343
*/
4444
public function __construct(Definitions $definitions, ProcessRequest $instance, ProcessRequestToken $token, array $data, $attemptNum = 1)
@@ -136,7 +136,7 @@ public function action(ProcessRequestToken $token = null, ServiceTaskInterface $
136136
$token->logError($modifiedException, $element);
137137

138138
Log::error('Service task failed: ' . $implementation . ' - ' . $message);
139-
Log::error($exception->getTraceAsString());
139+
Log::debug($exception->getTraceAsString());
140140
}
141141
}
142142

ProcessMaker/Models/ProcessRequest.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -586,7 +586,8 @@ public function logError(Throwable $exception, FlowElementInterface $element = n
586586
$errors[] = $error;
587587
$this->errors = $errors;
588588
$this->status = 'ERROR';
589-
Log::error($exception);
589+
Log::error($exception->getMessage());
590+
Log::debug($exception->getTraceAsString());
590591
if (!$this->isNonPersistent()) {
591592
$this->save();
592593
// Update Case status

0 commit comments

Comments
 (0)