New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Modify rule S5334: Add Spring example #3764

Open

hendrik-buchwald-sonarsource wants to merge 5 commits into master from appsec-1590-spel

Contributor

hendrik-buchwald-sonarsource commented Mar 14, 2024 •

edited

Implementation ticket: https://sonarsource.atlassian.net/browse/SONARSEC-5007

Review

A dedicated reviewer checked the rule description successfully for:

logical errors and incorrect information
information gaps and missing content
text style and tone
PR summary and labels follow the guidelines


          Add Spring example

964af2b

hendrik-buchwald-sonarsource added the java label

hendrik-buchwald-sonarsource added 4 commits

March 14, 2024 17:53


          Use different diff-id

39162d5


          Fix compliant code sample

7c0dd2d


          Add CWE

c52a006


          Fix metadata

24b0d74

sonarqube-next bot commented Mar 15, 2024

Quality Gate passed for 'rspec-frontend'

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

sonarqube-next bot commented Mar 15, 2024

Quality Gate passed for 'rspec-tools'

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

daniel-teuchert-sonarsource approved these changes

View reviewed changes

Contributor

daniel-teuchert-sonarsource left a comment

LGTM!

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