Skip to content

SONARJAVA-3884: Update rules metadata#3668

Merged
alban-auzeill merged 2 commits intomasterfrom
SONARJAVA-3884
Jun 25, 2021
Merged

SONARJAVA-3884: Update rules metadata#3668
alban-auzeill merged 2 commits intomasterfrom
SONARJAVA-3884

Conversation

@dorian-burihabwa-sonarsource
Copy link
Copy Markdown
Contributor

No description provided.

@@ -1,5 +1,5 @@
<p>Formatted SQL queries can be difficult to maintain, debug and can increase the risk of SQL injection when concatenating untrusted values into the
query. However, this rule doesn’t detect SQL injections (unlike rule {rule:javasecurity:S3649}), the goal is only to highlight complex/formatted queries.</p>
query. However, this rule doesn’t detect SQL injections (unlike rule {rule:java:S3649}), the goal is only to highlight complex/formatted queries.</p>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

S3649 does not exist in SonarJava but only in SonarSecurity.

@@ -1,204 +1,93 @@
<p><a href="https://www.w3.org/TR/xml/">XML specification</a> allows the use of entities that can be <a
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We should not delete this file! S2755 is a SonarJava rule.

@dorian-burihabwa-sonarsource dorian-burihabwa-sonarsource changed the title SONARJAVA-3884: Update rule metadata SONARJAVA-3884: Update rules metadata Jun 25, 2021
* Revert link change in S2077
* Add missing description of S2755
@sonarsource-next
Copy link
Copy Markdown

SonarQube Quality Gate

Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@dorian-burihabwa-sonarsource dorian-burihabwa-sonarsource marked this pull request as ready for review June 25, 2021 11:47
@alban-auzeill alban-auzeill merged commit 4431332 into master Jun 25, 2021
@alban-auzeill alban-auzeill deleted the SONARJAVA-3884 branch June 25, 2021 11:56
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.

2 participants