Skip to content

Commit b841ee3

Browse files
authored
Add several Sonar hotspot mappings (#486)
1 parent 394739f commit b841ee3

File tree

122 files changed

+1183
-75
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

122 files changed

+1183
-75
lines changed

.github/workflows/checks.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ jobs:
6464
python-version: '3.11'
6565

6666
- name: Install Semgrep
67-
run: python3 -m pip install semgrep
67+
run: python3 -m pip install semgrep==1.84.1
6868

6969
- name: Run Check task
7070
uses: gradle/gradle-build-action@842c587ad8aa4c68eeba24c396e15af4c2e9f30a # v2.9.0

.github/workflows/release.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ jobs:
2828
python-version: '3.11'
2929

3030
- name: Install Semgrep
31-
run: python3 -m pip install semgrep
31+
run: python3 -m pip install semgrep==1.84.1
3232

3333
- uses: actions/setup-java@v3
3434
with:

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ Follow these instructions if you intend to modify and build this project from so
2727
[here](https://semgrep.dev/docs/getting-started/#installing-and-running-semgrep-locally)
2828
for instructions. It can usually be done via `pip`:
2929
```shell
30-
pip install semgrep
30+
pip install semgrep==1.84.1
3131
```
3232

3333
If your python library paths contain your home directory as a root folder (i.e.

core-codemods/src/intTest/java/io/codemodder/integration/GitRepositoryTest.java

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,12 @@ protected void verifyNoFailedFiles(final CodeTFReport report) {
9797
.map(CodeTFResult::getFailedFiles)
9898
.flatMap(Collection::stream)
9999
.toList();
100-
assertThat(failedFiles.size(), is(0));
100+
if (!failedFiles.isEmpty()) {
101+
System.out.println("Failed files during scan:");
102+
failedFiles.forEach(System.err::println);
103+
}
104+
int size = failedFiles.size();
105+
assertThat(size, is(0));
101106
}
102107

103108
protected void verifyStandardCodemodResults(final List<CodeTFChangesetEntry> fileChanges) {

core-codemods/src/main/java/io/codemodder/codemods/DefaultCodemods.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
import io.codemodder.codemods.semgrep.SemgrepServletResponseWriterXSSCodemod;
1313
import io.codemodder.codemods.semgrep.SemgrepWeakRandomCodemod;
1414
import io.codemodder.codemods.semgrep.SemgrepXXECodemod;
15+
import io.codemodder.codemods.sonar.*;
1516
import java.util.List;
1617

1718
/**
@@ -89,12 +90,15 @@ public static List<Class<? extends CodeChanger>> asList() {
8990
SemgrepOverlyPermissiveFilePermissionsCodemod.class,
9091
SimplifyRestControllerAnnotationsCodemod.class,
9192
SubstituteReplaceAllCodemod.class,
93+
SonarCookieMissingSecureFlagCodemod.class,
9294
SonarJNDIInjectionCodemod.class,
9395
SonarObjectDeserializationCodemod.class,
9496
SonarRemoveUnthrowableExceptionCodemod.class,
9597
SonarSQLInjectionCodemod.class,
9698
SonarSSRFCodemod.class,
9799
SonarUnsafeReflectionRemediationCodemod.class,
100+
SonarWeakHashingAlgorithmCodemod.class,
101+
SonarWeakRandomCodemod.class,
98102
SonarXXECodemod.class,
99103
SQLParameterizerCodemod.class,
100104
SSRFCodemod.class,

core-codemods/src/main/java/io/codemodder/codemods/AddMissingOverrideCodemod.java renamed to core-codemods/src/main/java/io/codemodder/codemods/sonar/AddMissingOverrideCodemod.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
package io.codemodder.codemods;
1+
package io.codemodder.codemods.sonar;
22

33
import com.github.javaparser.ast.CompilationUnit;
44
import com.github.javaparser.ast.body.MethodDeclaration;

core-codemods/src/main/java/io/codemodder/codemods/AvoidImplicitPublicConstructorCodemod.java renamed to core-codemods/src/main/java/io/codemodder/codemods/sonar/AvoidImplicitPublicConstructorCodemod.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
package io.codemodder.codemods;
1+
package io.codemodder.codemods.sonar;
22

33
import com.github.javaparser.ast.CompilationUnit;
44
import com.github.javaparser.ast.Modifier;

core-codemods/src/main/java/io/codemodder/codemods/ConstantNameStringGenerator.java renamed to core-codemods/src/main/java/io/codemodder/codemods/sonar/ConstantNameStringGenerator.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
package io.codemodder.codemods;
1+
package io.codemodder.codemods.sonar;
22

33
import java.util.Set;
44
import java.util.regex.Matcher;

core-codemods/src/main/java/io/codemodder/codemods/CreateConstantForLiteral.java renamed to core-codemods/src/main/java/io/codemodder/codemods/sonar/CreateConstantForLiteral.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
package io.codemodder.codemods;
1+
package io.codemodder.codemods.sonar;
22

33
import com.github.javaparser.ast.CompilationUnit;
44
import com.github.javaparser.ast.Modifier;

core-codemods/src/main/java/io/codemodder/codemods/DeclareVariableOnSeparateLine.java renamed to core-codemods/src/main/java/io/codemodder/codemods/sonar/DeclareVariableOnSeparateLine.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
package io.codemodder.codemods;
1+
package io.codemodder.codemods.sonar;
22

33
import com.github.javaparser.ast.Node;
44
import com.github.javaparser.ast.NodeList;

0 commit comments

Comments
 (0)