Skip to content

SONARXML-264 Fix FN on S2068 in web.config of .NET Web Applications#363

Merged
pavel-mikula-sonarsource merged 5 commits intomasterfrom
tt/USER-1192
Nov 26, 2025
Merged

SONARXML-264 Fix FN on S2068 in web.config of .NET Web Applications#363
pavel-mikula-sonarsource merged 5 commits intomasterfrom
tt/USER-1192

Conversation

@tomasz-tylenda-sonarsource
Copy link
Copy Markdown
Contributor

@tomasz-tylenda-sonarsource tomasz-tylenda-sonarsource commented Nov 4, 2025

@sonarqube-next
Copy link
Copy Markdown

sonarqube-next bot commented Nov 5, 2025

Quality Gate failed Quality Gate failed

Failed conditions
1 New issue

See analysis details on SonarQube

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE SonarQube for IDE

@github-actions
Copy link
Copy Markdown
Contributor

This PR is stale because it has been open 7 days with no activity. If there is no activity in the next 7 days it will be closed automatically

@github-actions github-actions bot added the stale label Nov 13, 2025
@tomasz-tylenda-sonarsource
Copy link
Copy Markdown
Contributor Author

Comment to avoid auto-closing

@github-actions github-actions bot removed the stale label Nov 14, 2025
@tomasz-tylenda-sonarsource
Copy link
Copy Markdown
Contributor Author

Still working on it.

Copy link
Copy Markdown
Contributor

@pavel-mikula-sonarsource pavel-mikula-sonarsource left a comment

Choose a reason for hiding this comment

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

The hardcoded "password" literal is weird

Copy link
Copy Markdown
Contributor

@pavel-mikula-sonarsource pavel-mikula-sonarsource left a comment

Choose a reason for hiding this comment

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

The hardcoded "password" literal is weird

…sCheck/app-settings/web.config

Co-authored-by: Pavel Mikula <57188685+pavel-mikula-sonarsource@users.noreply.github.com>
* More tests.
* Message.
@tomasz-tylenda-sonarsource
Copy link
Copy Markdown
Contributor Author

The hardcoded "password" literal is weird

I followed what what asked in USER-1192 ticket. It's hard for me to decide what makes sense and what does not. The ticket contains (in the comments) what the customer had in mind.

@sonarqube-next
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

@pavel-mikula-sonarsource pavel-mikula-sonarsource left a comment

Choose a reason for hiding this comment

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

LGTM

@pavel-mikula-sonarsource pavel-mikula-sonarsource merged commit 6957cc7 into master Nov 26, 2025
14 checks passed
@pavel-mikula-sonarsource pavel-mikula-sonarsource deleted the tt/USER-1192 branch November 26, 2025 13:57
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