From 735ce7340358fe9c812ce8a67b53cd15f9da9d12 Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" Date: Wed, 4 Nov 2026 12:12:00 +0000 Subject: [PATCH] Update ruling results MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 🤖 Generated with GitHub Actions --- .../commons-beanutils/java-S106.sarif | 109 ++-- .../commons-beanutils/java-S9438.sarif | 26 +- .../expected/eclipse-jetty/java-S3626.sarif | 6 +- .../resources/expected/guava/java-S112.sarif | 486 ++++++++++++++++-- .../resources/expected/guava/java-S3626.sarif | 4 +- 5 files changed, 522 insertions(+), 109 deletions(-) diff --git a/its/ruling/src/test/resources/expected/commons-beanutils/java-S106.sarif b/its/ruling/src/test/resources/expected/commons-beanutils/java-S106.sarif index 38abb634e5a..48e0f83aad3 100644 --- a/its/ruling/src/test/resources/expected/commons-beanutils/java-S106.sarif +++ b/its/ruling/src/test/resources/expected/commons-beanutils/java-S106.sarif @@ -1,43 +1,66 @@ -{ - "$schema" : "https://json.schemastore.org/sarif-2.1.0.json", - "version" : "2.1.0", - "runs" : [ { - "tool" : { - "driver" : { - "name" : "LITS", - "language" : "en-US", - "contents" : [ "localizedData", "nonLocalizedData" ], - "isComprehensive" : false - } - }, - "language" : "en-US", - "results" : [ { - "ruleId" : "java:S106", - "ruleIndex" : -1, - "kind" : "fail", - "level" : "warning", - "message" : { - "text" : "Replace this use of System.out by a logger." - }, - "locations" : [ { - "id" : -1, - "physicalLocation" : { - "artifactLocation" : { - "uri" : "commons-beanutils:src/main/java/org/apache/commons/beanutils2/BeanMap.java", - "index" : -1 - }, - "region" : { - "startLine" : 855, - "startColumn" : 9, - "endLine" : 855, - "endColumn" : 19 - } - } - } ], - "rank" : -1.0 - } - ], - "newlineSequences" : [ "\r\n", "\n" ], - "columnKind" : "utf16CodeUnits" - } ] -} +{ + "$schema" : "https://json.schemastore.org/sarif-2.1.0.json", + "version" : "2.1.0", + "runs" : [ { + "tool" : { + "driver" : { + "name" : "LITS", + "language" : "en-US", + "contents" : [ "localizedData", "nonLocalizedData" ], + "isComprehensive" : false + } + }, + "language" : "en-US", + "results" : [ { + "ruleId" : "java:S106", + "ruleIndex" : -1, + "kind" : "fail", + "level" : "warning", + "message" : { + "text" : "Replace this use of System.out by a logger." + }, + "locations" : [ { + "id" : -1, + "physicalLocation" : { + "artifactLocation" : { + "uri" : "commons-beanutils:src/main/java/org/apache/commons/beanutils2/BeanMap.java", + "index" : -1 + }, + "region" : { + "startLine" : 855, + "startColumn" : 9, + "endLine" : 855, + "endColumn" : 19 + } + } + } ], + "rank" : -1.0 + }, { + "ruleId" : "java:S106", + "ruleIndex" : -1, + "kind" : "fail", + "level" : "warning", + "message" : { + "text" : "Replace this use of System.out by a logger." + }, + "locations" : [ { + "id" : -1, + "physicalLocation" : { + "artifactLocation" : { + "uri" : "commons-beanutils:src/main/java/org/apache/commons/beanutils2/BeanMap.java", + "index" : -1 + }, + "region" : { + "startLine" : 866, + "startColumn" : 9, + "endLine" : 866, + "endColumn" : 19 + } + } + } ], + "rank" : -1.0 + } ], + "newlineSequences" : [ "\r\n", "\n" ], + "columnKind" : "utf16CodeUnits" + } ] +} \ No newline at end of file diff --git a/its/ruling/src/test/resources/expected/commons-beanutils/java-S9438.sarif b/its/ruling/src/test/resources/expected/commons-beanutils/java-S9438.sarif index 4d76fdeee81..0554e822c33 100644 --- a/its/ruling/src/test/resources/expected/commons-beanutils/java-S9438.sarif +++ b/its/ruling/src/test/resources/expected/commons-beanutils/java-S9438.sarif @@ -59,30 +59,6 @@ } } ], "rank" : -1.0 - }, { - "ruleId" : "java:S9438", - "ruleIndex" : -1, - "kind" : "fail", - "level" : "warning", - "message" : { - "text" : "Make this private field \"final\", it is never reassigned." - }, - "locations" : [ { - "id" : -1, - "physicalLocation" : { - "artifactLocation" : { - "uri" : "commons-beanutils:src/main/java/org/apache/commons/beanutils2/BeanMap.java", - "index" : -1 - }, - "region" : { - "startLine" : 42, - "startColumn" : 5, - "endLine" : 42, - "endColumn" : 7 - } - } - } ], - "rank" : -1.0 }, { "ruleId" : "java:S9438", "ruleIndex" : -1, @@ -111,4 +87,4 @@ "newlineSequences" : [ "\r\n", "\n" ], "columnKind" : "utf16CodeUnits" } ] -} +} \ No newline at end of file diff --git a/its/ruling/src/test/resources/expected/eclipse-jetty/java-S3626.sarif b/its/ruling/src/test/resources/expected/eclipse-jetty/java-S3626.sarif index 64ed6742f7c..ed27a807037 100644 --- a/its/ruling/src/test/resources/expected/eclipse-jetty/java-S3626.sarif +++ b/its/ruling/src/test/resources/expected/eclipse-jetty/java-S3626.sarif @@ -17,7 +17,7 @@ "kind" : "fail", "level" : "warning", "message" : { - "text" : "Remove this redundant return." + "text" : "Remove this redundant jump." }, "locations" : [ { "id" : -1, @@ -41,7 +41,7 @@ "kind" : "fail", "level" : "warning", "message" : { - "text" : "Remove this redundant continue." + "text" : "Remove this redundant jump." }, "locations" : [ { "id" : -1, @@ -65,7 +65,7 @@ "kind" : "fail", "level" : "warning", "message" : { - "text" : "Remove this redundant return." + "text" : "Remove this redundant jump." }, "locations" : [ { "id" : -1, diff --git a/its/ruling/src/test/resources/expected/guava/java-S112.sarif b/its/ruling/src/test/resources/expected/guava/java-S112.sarif index 7d80b027f65..59c02a89cdc 100644 --- a/its/ruling/src/test/resources/expected/guava/java-S112.sarif +++ b/its/ruling/src/test/resources/expected/guava/java-S112.sarif @@ -1,36 +1,450 @@ -{ - "$schema": "https://json.schemastore.org/sarif-2.1.0.json", - "version": "2.1.0", - "runs": [ - { - "tool": { - "driver": { - "name": "LITS" - } - }, - "results": [ - { - "ruleId": "java:S112", - "message": { - "text": "RC-451 smoke test: define and throw a dedicated exception instead of using a generic one." - }, - "locations": [ - { - "physicalLocation": { - "artifactLocation": { - "uri": "guava:src/com/google/common/cache/CacheLoader.java" - }, - "region": { - "startLine": 69, - "startColumn": 5, - "endLine": 69, - "endColumn": 10 - } - } - } - ] - } - ] - } - ] -} +{ + "$schema" : "https://json.schemastore.org/sarif-2.1.0.json", + "version" : "2.1.0", + "runs" : [ { + "tool" : { + "driver" : { + "name" : "LITS", + "language" : "en-US", + "contents" : [ "localizedData", "nonLocalizedData" ], + "isComprehensive" : false + } + }, + "language" : "en-US", + "results" : [ { + "ruleId" : "java:S112", + "ruleIndex" : -1, + "kind" : "fail", + "level" : "warning", + "message" : { + "text" : "Replace generic exceptions with specific library exceptions or a custom exception." + }, + "locations" : [ { + "id" : -1, + "physicalLocation" : { + "artifactLocation" : { + "uri" : "guava:src/com/google/common/cache/CacheLoader.java", + "index" : -1 + }, + "region" : { + "startLine" : 69, + "startColumn" : 40, + "endLine" : 69, + "endColumn" : 49 + } + } + } ], + "rank" : -1.0 + }, { + "ruleId" : "java:S112", + "ruleIndex" : -1, + "kind" : "fail", + "level" : "warning", + "message" : { + "text" : "Replace generic exceptions with specific library exceptions or a custom exception." + }, + "locations" : [ { + "id" : -1, + "physicalLocation" : { + "artifactLocation" : { + "uri" : "guava:src/com/google/common/cache/CacheLoader.java", + "index" : -1 + }, + "region" : { + "startLine" : 121, + "startColumn" : 63, + "endLine" : 121, + "endColumn" : 72 + } + } + } ], + "rank" : -1.0 + }, { + "ruleId" : "java:S112", + "ruleIndex" : -1, + "kind" : "fail", + "level" : "warning", + "message" : { + "text" : "Replace generic exceptions with specific library exceptions or a custom exception." + }, + "locations" : [ { + "id" : -1, + "physicalLocation" : { + "artifactLocation" : { + "uri" : "guava:src/com/google/common/reflect/AbstractInvocationHandler.java", + "index" : -1 + }, + "region" : { + "startLine" : 99, + "startColumn" : 14, + "endLine" : 99, + "endColumn" : 23 + } + } + } ], + "rank" : -1.0 + }, { + "ruleId" : "java:S112", + "ruleIndex" : -1, + "kind" : "fail", + "level" : "warning", + "message" : { + "text" : "Replace generic exceptions with specific library exceptions or a custom exception." + }, + "locations" : [ { + "id" : -1, + "physicalLocation" : { + "artifactLocation" : { + "uri" : "guava:src/com/google/common/util/concurrent/AbstractExecutionThreadService.java", + "index" : -1 + }, + "region" : { + "startLine" : 100, + "startColumn" : 35, + "endLine" : 100, + "endColumn" : 44 + } + } + } ], + "rank" : -1.0 + }, { + "ruleId" : "java:S112", + "ruleIndex" : -1, + "kind" : "fail", + "level" : "warning", + "message" : { + "text" : "Replace generic exceptions with specific library exceptions or a custom exception." + }, + "locations" : [ { + "id" : -1, + "physicalLocation" : { + "artifactLocation" : { + "uri" : "guava:src/com/google/common/util/concurrent/AbstractExecutionThreadService.java", + "index" : -1 + }, + "region" : { + "startLine" : 116, + "startColumn" : 40, + "endLine" : 116, + "endColumn" : 49 + } + } + } ], + "rank" : -1.0 + }, { + "ruleId" : "java:S112", + "ruleIndex" : -1, + "kind" : "fail", + "level" : "warning", + "message" : { + "text" : "Replace generic exceptions with specific library exceptions or a custom exception." + }, + "locations" : [ { + "id" : -1, + "physicalLocation" : { + "artifactLocation" : { + "uri" : "guava:src/com/google/common/util/concurrent/AbstractExecutionThreadService.java", + "index" : -1 + }, + "region" : { + "startLine" : 124, + "startColumn" : 36, + "endLine" : 124, + "endColumn" : 45 + } + } + } ], + "rank" : -1.0 + }, { + "ruleId" : "java:S112", + "ruleIndex" : -1, + "kind" : "fail", + "level" : "warning", + "message" : { + "text" : "Replace generic exceptions with specific library exceptions or a custom exception." + }, + "locations" : [ { + "id" : -1, + "physicalLocation" : { + "artifactLocation" : { + "uri" : "guava:src/com/google/common/util/concurrent/AbstractIdleService.java", + "index" : -1 + }, + "region" : { + "startLine" : 81, + "startColumn" : 44, + "endLine" : 81, + "endColumn" : 53 + } + } + } ], + "rank" : -1.0 + }, { + "ruleId" : "java:S112", + "ruleIndex" : -1, + "kind" : "fail", + "level" : "warning", + "message" : { + "text" : "Replace generic exceptions with specific library exceptions or a custom exception." + }, + "locations" : [ { + "id" : -1, + "physicalLocation" : { + "artifactLocation" : { + "uri" : "guava:src/com/google/common/util/concurrent/AbstractIdleService.java", + "index" : -1 + }, + "region" : { + "startLine" : 84, + "startColumn" : 45, + "endLine" : 84, + "endColumn" : 54 + } + } + } ], + "rank" : -1.0 + }, { + "ruleId" : "java:S112", + "ruleIndex" : -1, + "kind" : "fail", + "level" : "warning", + "message" : { + "text" : "Replace generic exceptions with specific library exceptions or a custom exception." + }, + "locations" : [ { + "id" : -1, + "physicalLocation" : { + "artifactLocation" : { + "uri" : "guava:src/com/google/common/util/concurrent/AbstractScheduledService.java", + "index" : -1 + }, + "region" : { + "startLine" : 265, + "startColumn" : 52, + "endLine" : 265, + "endColumn" : 61 + } + } + } ], + "rank" : -1.0 + }, { + "ruleId" : "java:S112", + "ruleIndex" : -1, + "kind" : "fail", + "level" : "warning", + "message" : { + "text" : "Replace generic exceptions with specific library exceptions or a custom exception." + }, + "locations" : [ { + "id" : -1, + "physicalLocation" : { + "artifactLocation" : { + "uri" : "guava:src/com/google/common/util/concurrent/AbstractScheduledService.java", + "index" : -1 + }, + "region" : { + "startLine" : 272, + "startColumn" : 35, + "endLine" : 272, + "endColumn" : 44 + } + } + } ], + "rank" : -1.0 + }, { + "ruleId" : "java:S112", + "ruleIndex" : -1, + "kind" : "fail", + "level" : "warning", + "message" : { + "text" : "Replace generic exceptions with specific library exceptions or a custom exception." + }, + "locations" : [ { + "id" : -1, + "physicalLocation" : { + "artifactLocation" : { + "uri" : "guava:src/com/google/common/util/concurrent/AbstractScheduledService.java", + "index" : -1 + }, + "region" : { + "startLine" : 279, + "startColumn" : 36, + "endLine" : 279, + "endColumn" : 45 + } + } + } ], + "rank" : -1.0 + }, { + "ruleId" : "java:S112", + "ruleIndex" : -1, + "kind" : "fail", + "level" : "warning", + "message" : { + "text" : "Replace generic exceptions with specific library exceptions or a custom exception." + }, + "locations" : [ { + "id" : -1, + "physicalLocation" : { + "artifactLocation" : { + "uri" : "guava:src/com/google/common/util/concurrent/AbstractScheduledService.java", + "index" : -1 + }, + "region" : { + "startLine" : 568, + "startColumn" : 58, + "endLine" : 568, + "endColumn" : 67 + } + } + } ], + "rank" : -1.0 + }, { + "ruleId" : "java:S112", + "ruleIndex" : -1, + "kind" : "fail", + "level" : "warning", + "message" : { + "text" : "Replace generic exceptions with specific library exceptions or a custom exception." + }, + "locations" : [ { + "id" : -1, + "physicalLocation" : { + "artifactLocation" : { + "uri" : "guava:src/com/google/common/util/concurrent/AsyncFunction.java", + "index" : -1 + }, + "region" : { + "startLine" : 43, + "startColumn" : 55, + "endLine" : 43, + "endColumn" : 64 + } + } + } ], + "rank" : -1.0 + }, { + "ruleId" : "java:S112", + "ruleIndex" : -1, + "kind" : "fail", + "level" : "warning", + "message" : { + "text" : "Replace generic exceptions with specific library exceptions or a custom exception." + }, + "locations" : [ { + "id" : -1, + "physicalLocation" : { + "artifactLocation" : { + "uri" : "guava:src/com/google/common/util/concurrent/FutureFallback.java", + "index" : -1 + }, + "region" : { + "startLine" : 51, + "startColumn" : 50, + "endLine" : 51, + "endColumn" : 59 + } + } + } ], + "rank" : -1.0 + }, { + "ruleId" : "java:S112", + "ruleIndex" : -1, + "kind" : "fail", + "level" : "warning", + "message" : { + "text" : "Replace generic exceptions with specific library exceptions or a custom exception." + }, + "locations" : [ { + "id" : -1, + "physicalLocation" : { + "artifactLocation" : { + "uri" : "guava:src/com/google/common/util/concurrent/Futures.java", + "index" : -1 + }, + "region" : { + "startLine" : 793, + "startColumn" : 62, + "endLine" : 793, + "endColumn" : 71 + } + } + } ], + "rank" : -1.0 + }, { + "ruleId" : "java:S112", + "ruleIndex" : -1, + "kind" : "fail", + "level" : "warning", + "message" : { + "text" : "Replace generic exceptions with specific library exceptions or a custom exception." + }, + "locations" : [ { + "id" : -1, + "physicalLocation" : { + "artifactLocation" : { + "uri" : "guava:src/com/google/common/util/concurrent/Futures.java", + "index" : -1 + }, + "region" : { + "startLine" : 1414, + "startColumn" : 60, + "endLine" : 1414, + "endColumn" : 69 + } + } + } ], + "rank" : -1.0 + }, { + "ruleId" : "java:S112", + "ruleIndex" : -1, + "kind" : "fail", + "level" : "warning", + "message" : { + "text" : "Replace generic exceptions with specific library exceptions or a custom exception." + }, + "locations" : [ { + "id" : -1, + "physicalLocation" : { + "artifactLocation" : { + "uri" : "guava:src/com/google/common/util/concurrent/SimpleTimeLimiter.java", + "index" : -1 + }, + "region" : { + "startLine" : 148, + "startColumn" : 14, + "endLine" : 148, + "endColumn" : 23 + } + } + } ], + "rank" : -1.0 + }, { + "ruleId" : "java:S112", + "ruleIndex" : -1, + "kind" : "fail", + "level" : "warning", + "message" : { + "text" : "Replace generic exceptions with specific library exceptions or a custom exception." + }, + "locations" : [ { + "id" : -1, + "physicalLocation" : { + "artifactLocation" : { + "uri" : "guava:src/com/google/common/util/concurrent/TimeLimiter.java", + "index" : -1 + }, + "region" : { + "startLine" : 105, + "startColumn" : 59, + "endLine" : 105, + "endColumn" : 68 + } + } + } ], + "rank" : -1.0 + } ], + "newlineSequences" : [ "\r\n", "\n" ], + "columnKind" : "utf16CodeUnits" + } ] +} \ No newline at end of file diff --git a/its/ruling/src/test/resources/expected/guava/java-S3626.sarif b/its/ruling/src/test/resources/expected/guava/java-S3626.sarif index 6455036ab75..73c44c3fb12 100644 --- a/its/ruling/src/test/resources/expected/guava/java-S3626.sarif +++ b/its/ruling/src/test/resources/expected/guava/java-S3626.sarif @@ -17,7 +17,7 @@ "kind" : "fail", "level" : "warning", "message" : { - "text" : "Remove this redundant continue." + "text" : "Remove this redundant jump." }, "locations" : [ { "id" : -1, @@ -41,7 +41,7 @@ "kind" : "fail", "level" : "warning", "message" : { - "text" : "Remove this redundant continue." + "text" : "Remove this redundant jump." }, "locations" : [ { "id" : -1,