Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

SONARJAVA-3884: Update rules metadata#3668

Merged
alban-auzeill merged 2 commits into
masterSonarSource/sonar-java:masterfrom
SONARJAVA-3884SonarSource/sonar-java:SONARJAVA-3884Copy head branch name to clipboard
Jun 25, 2021
Merged

SONARJAVA-3884: Update rules metadata#3668
alban-auzeill merged 2 commits into
masterSonarSource/sonar-java:masterfrom
SONARJAVA-3884SonarSource/sonar-java:SONARJAVA-3884Copy head branch name to clipboard

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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.