Skip to content

feat: adding new vulnerabilities - #28

Open
dylansnyk wants to merge 19 commits into
mainfrom
new-vulns
Open

feat: adding new vulnerabilities#28
dylansnyk wants to merge 19 commits into
mainfrom
new-vulns

Conversation

@dylansnyk

Copy link
Copy Markdown
Owner

No description provided.

Comment thread src/main/java/io/snyk/dylansnyk/demo/controller/EchoController.java
Comment thread src/main/java/io/snyk/dylansnyk/demo/controller/EchoController.java
Repository owner deleted a comment from snyk-io Bot Apr 9, 2025
@dylansnyk

dylansnyk commented Jul 22, 2025

Copy link
Copy Markdown
Owner Author

Snyk checks have failed. 1 issues have been found so far.

Icon Severity Issues
Critical 0
High 1
Medium 0
Low 0

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

code/snyk check is complete. 1 issues have been found. (View Details)

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

public void makeRequest(@RequestParam String url) {
HttpClient client = HttpClient.newHttpClient();

HttpRequest request = HttpRequest.newBuilder()

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  Server-Side Request Forgery (SSRF)

Unsanitized input from an HTTP parameter flows into uri, where it is used as an URL to perform a request. This may result in a Server-Side Request Forgery vulnerability.

Line 44 | CWE-918 | Priority score 834 | Learn more about this vulnerability
Data flow: 5 steps

Step 1 - 2

public void makeRequest(@RequestParam String url) {

Step 3 - 4 src/main/java/io/snyk/dylansnyk/demo/controller/EchoController.java#L45

Step 5

HttpRequest request = HttpRequest.newBuilder()

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant