From 6670a3d360c3c7a50c067da35f829cd4e7c635c1 Mon Sep 17 00:00:00 2001 From: Ryan Dens Date: Thu, 18 Jul 2024 15:50:49 -0400 Subject: [PATCH] :bug: Defensive null check (#424) - **:recycle: provide messageText to all RuleSarifFactory impls** - **Bind AppScan sarif to rule by rule name from message text** - **:bug: fix distinct rule ID bug** - **:bug: defensive null check** --- .../src/main/java/io/codemodder/DefaultSarifParser.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/framework/codemodder-base/src/main/java/io/codemodder/DefaultSarifParser.java b/framework/codemodder-base/src/main/java/io/codemodder/DefaultSarifParser.java index 8bf5f6546..d1a6ba370 100644 --- a/framework/codemodder-base/src/main/java/io/codemodder/DefaultSarifParser.java +++ b/framework/codemodder-base/src/main/java/io/codemodder/DefaultSarifParser.java @@ -64,7 +64,8 @@ private RuleDescriptor extractRuleId(final Result result, final Run run) { return null; } } - return new RuleDescriptor(result.getRuleId(), result.getMessage().getText()); + return new RuleDescriptor( + result.getRuleId(), result.getMessage() != null ? result.getMessage().getText() : null); } private Stream> fromSarif(