From 229c81ac8c0e20f4b37548b93465c9a84e636712 Mon Sep 17 00:00:00 2001 From: Nico Rosner Date: Tue, 10 May 2022 16:50:36 -0400 Subject: [PATCH 1/2] Remap SARIF severity levels for consistency with GitHub Action --- .../amazonaws/gurureviewercli/adapter/ResultsAdapter.java | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/main/java/com/amazonaws/gurureviewercli/adapter/ResultsAdapter.java b/src/main/java/com/amazonaws/gurureviewercli/adapter/ResultsAdapter.java index 7f3b2f0..def1e94 100644 --- a/src/main/java/com/amazonaws/gurureviewercli/adapter/ResultsAdapter.java +++ b/src/main/java/com/amazonaws/gurureviewercli/adapter/ResultsAdapter.java @@ -211,13 +211,11 @@ private static String getSarifSeverity(RecommendationSummary recommendation) { } switch (recommendation.severity()) { case INFO: - return Result.Level.NONE.value(); case LOW: - return Result.Level.NONE.value(); + return Result.Level.NOTE.value(); case MEDIUM: - return Result.Level.NONE.value(); - case HIGH: return Result.Level.WARNING.value(); + case HIGH: case CRITICAL: return Result.Level.ERROR.value(); default: From 5203c3021ab72d56abfc93ac9ea18173e8378e7a Mon Sep 17 00:00:00 2001 From: Nico Rosner Date: Tue, 10 May 2022 17:01:15 -0400 Subject: [PATCH 2/2] Remap High to WARNING instead of CRITICAL --- .../com/amazonaws/gurureviewercli/adapter/ResultsAdapter.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main/java/com/amazonaws/gurureviewercli/adapter/ResultsAdapter.java b/src/main/java/com/amazonaws/gurureviewercli/adapter/ResultsAdapter.java index def1e94..15ba58f 100644 --- a/src/main/java/com/amazonaws/gurureviewercli/adapter/ResultsAdapter.java +++ b/src/main/java/com/amazonaws/gurureviewercli/adapter/ResultsAdapter.java @@ -214,8 +214,8 @@ private static String getSarifSeverity(RecommendationSummary recommendation) { case LOW: return Result.Level.NOTE.value(); case MEDIUM: - return Result.Level.WARNING.value(); case HIGH: + return Result.Level.WARNING.value(); case CRITICAL: return Result.Level.ERROR.value(); default: