diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
index de05388..efa4594 100644
--- a/.github/workflows/build.yml
+++ b/.github/workflows/build.yml
@@ -13,7 +13,7 @@ jobs:
- uses: actions/checkout@v3
with:
fetch-depth: 0 # Shallow clones should be disabled for a better relevancy of analysis
- - name: Set up JDK 11
+ - name: Set up JDK 17
uses: actions/setup-java@v3
with:
java-version: 17
@@ -34,4 +34,4 @@ jobs:
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # Needed to get PR information, if any
SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
- run: mvn -B verify org.sonarsource.scanner.maven:sonar-maven-plugin:sonar -Dsonar.projectKey=SonarSourceResearch_DeepSAST_Demo
+ run: mvn -B verify org.sonarsource.scanner.maven:sonar-maven-plugin:sonar -Dsonar.projectKey=abordes96_deeper-sast-demo
diff --git a/README.md b/README.md
index 75548fd..fd81185 100644
--- a/README.md
+++ b/README.md
@@ -1,8 +1,28 @@
# deeper-sast-demo
-## Storyboard
+The goal of this demo is to show the Sonar deeper SAST capabilities of the Java engine. We want to highlight that the usage of external libraries can introduce severe security vulnerabilities, which can be detected by Sonar deeper SAST.
+
+## Setup instructions
+
+This repository is supposed to be added as a SonarCloud project for analysis via GitHub actions.
+
+* Fork this project *with all branches* (untick the default checkbox, "Copy the `main` branch only").
+* Go to the `Actions` tab of your forked repository and enable workflows by selecting `I understand my workflows, go ahead and enable them`.
+* Go to [sonarcloud.io](https://sonarcloud.io/sessions/new) and sign up with your GitHub account.
+* Create a new organization under your name if there is none.
+* Give SonarCloud permission to see the forked repository.
+* Add your repository as a new Project.
+* Go to `Administration` -> `Analysis Method` and uncheck `Automatic Analysis`.
+* Select `Set up analysis via other methods` -> `With GitHub Actions`.
+ * Add the displayed GitHub Secret to your repository.
+ * Update the `sonar.organization` value in the `pom.xml` file.
+ * Create a new file`.github/workflows/build.yml`, copy & paste the displayed content to it.
+ * Set the `java-version` in the `build.yml` file to `17`.
+* On your GitHub repository: Go to the `Pull requests` tab and create a new PR from the `introduce-user-migration-feature` and `allow-imports` branches to the `main` branch of your fork. Be careful that, by default, the PR targets the upstream repository.
+
+The first two issues will be displayed on the `main` branch and the other two issues on distinct Pull Requests.
-The goal of this demo is to show the DeepSAST capabilities of the Java engine. We want to highlight that the usage of external libraries can introduce severe security vulnerabilities, which can be detected by Deep SAST.
+## Storyboard
The demo is a fictive Spring application implementing different functionalities which are vulnerable to security issues detected by our engine. All of these issues contain at least one step where the data flow:
@@ -10,14 +30,14 @@ The demo is a fictive Spring application implementing different functionalities
2. passes through a library, or
3. ends in a dangerous sink within a library.
-Thus these issues are only detected because of the DeepSAST feature of the engine.
+Thus these issues are only detected because of the deeper SAST feature of the Sonar engine.
There are four issues: two of these are already committed to the main branch of the application. Additionally, there are two pending pull requests (PR), which each introduce another vulnerability. For these issues, the chosen examples aim to demonstrate that the proposed source code in the PR does not look dangerous or security-sensitive and would likely be merged.
### Issue 1 - Session Cookie Handling (main branch)
* Vulnerability Type: Deserialization ([S5135](https://rules.sonarsource.com/java/RSPEC-5135/))
-* DeepSAST Dataflow:
+* Deeper SAST Dataflow:
* Passthrough: `org.apache.commons.codec.binary.Base64.decodeBase64`
This vulnerability resides within the session cookie handling of the application. A vulnerability is introduced by deserializing user-controllable data from a header (`Session-Auth`), which can be exploited to execute arbitrary code. The data provided in the header is passed through the `decodeBase64` library function before being deserialized.
@@ -26,10 +46,10 @@ This vulnerability resides within the session cookie handling of the application
### Issue 2 - User Images (main branch)
* Vulnerability Type: Path Injection ([S2083](https://rules.sonarsource.com/java/RSPEC-2083/))
-* DeepSAST Dataflow:
- * Source: org.springframework.web.context.request.getRemoteUser
- * Passthrough: org.apache.tomcat.util.buf.UDecoder.URLDecode
- * Sink: cn.hutool.cache.file.LRUFileCache.getFileBytes
+* Deeper SAST Dataflow:
+ * Source: `org.springframework.web.context.request.getRemoteUser`
+ * Passthrough: `org.apache.tomcat.util.buf.UDecoder.URLDecode`
+ * Sink: `cn.hutool.cache.file.LRUFileCache.getFileBytes`
This vulnerability resides within the code responsible for retrieving user images. The library function `getRemoteUser` is used to retrieve the user-controllable username, which is passed through the `URLDecode` library function. The result is concatenated to a file path, which is passed to the `getFileBytes` library function introducing a path injection vulnerability.
@@ -37,33 +57,15 @@ This vulnerability resides within the code responsible for retrieving user image
### Issue 3 - User Migration (PR 1 - Introduce user migration feature)
* Vulnerability Type: SQL Injection ([S3649](https://rules.sonarsource.com/java/RSPEC-3649/))
-* DeepSAST Dataflow:
- * Sink: com.mysql.cj.jdbc.ConnectionImpl.setSavepoint
+* Deeper SAST Dataflow:
+ * Sink: `com.mysql.cj.jdbc.ConnectionImpl.setSavepoint`
This PR adds a feature to migrate users from the existing H2 database to MySQL. Although the proposed change does not seem to contain any vulnerabilities, the `setSavepoint` library function is vulnerable to SQL injection if the passed argument is user-controllable. Thus this PR introduces a critical vulnerability due to the usage of the unsafe library function.
### Issue 4 - XML User Import (PR 2 - Allow the import of users)
* Vulnerability Type: Deserialization ([S5135](https://rules.sonarsource.com/java/RSPEC-5135/))
-* DeepSAST Dataflow:
- * Sink: ca.odell.glazedlists.impl.io.BeanXMLByteCoder.decode
+* Deeper SAST Dataflow:
+ * Sink: `ca.odell.glazedlists.impl.io.BeanXMLByteCoder.decode`
-This PR adds a new feature to import users from an XML file. Although the code itself does not seem to contain any vulnerabilities, the `decode` library function is vulnerable to deserialzation if the passed argument is user-controllable. Thus this PR introduces a critical vulnerability due to the usage of the unsafe library function.
-
-
-## Setup instructions
-
-This repository is supposed to be added as a SonarCloud project for analysis via GitHub actions.
-
-* Fork this project *with all branches* (untick the default checkbox, "Copy the `main`` branch only").
-* Go to [sonarcloud.io](https://sonarcloud.io/sessions/new) and sign up with your GitHub account.
-* Create a new organization under your name if there is none.
-* Give SonarCloud permission to see the forked repository.
-* Add your repository as a new Project.
-* Go to `Administration` -> `Analysis Method` and uncheck `Automatic Analysis`.
-* Select `Set up analysis via other methods` -> `With GitHub Actions`.
- * Add the displayed GitHub Secret to your repository.
- * The `pom.xml` does not need to be adjusted.
- * Update the `.github/workflows/build.yml` file in the main branch with the displayed content and ensure that the `java-version` is set to `17`.
-
-The first two issues will be displayed on the `main` branch and the other two issues on distinct Pull Requests.
+This PR adds a new feature to import users from an XML file. Although the code itself does not seem to contain any vulnerabilities, the `decode` library function is vulnerable to deserialization if the passed argument is user-controllable. Thus this PR introduces a critical vulnerability due to the usage of the unsafe library function.
diff --git a/pom.xml b/pom.xml
index 7ca5e0c..9e94f8a 100644
--- a/pom.xml
+++ b/pom.xml
@@ -15,7 +15,7 @@
deeper-sast-demo
17
- sonarsourceresearch
+ test-mg
https://sonarcloud.io