Addressing feedback

elastic · Nov 14, 2022 · 5425543 · 5425543
1 parent b51f3f5
commit 5425543
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/x-pack/plugins/alerting/server/task_runner/execution_handler.test.ts b/x-pack/plugins/alerting/server/task_runner/execution_handler.test.ts
@@ -858,7 +858,7 @@ describe('Execution Handler', () => {
       `);
     });
 
-    it('does not populate the rule.url when the alert id is set to undefined', async () => {
+    it('does not populate the rule.url when the rule id is set to undefined', async () => {
       const execParams = {
         ...defaultExecutionParams,
         rule: ruleWithUrl,

diff --git a/x-pack/plugins/alerting/server/task_runner/execution_handler.ts b/x-pack/plugins/alerting/server/task_runner/execution_handler.ts
@@ -297,6 +297,9 @@ export class ExecutionHandler<
 
       return ruleUrl.toString();
     } catch (error) {
+      this.logger.debug(
+        `Rule "${this.rule.id}" encountered an error while constructing the rule.url variable: ${error.message}`
+      );
       return;
     }
   }