Skip to content
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.

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

Update SonarAnalyzer.CSharp to latest version #31

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
39 changes: 39 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,45 @@ and adheres to a project-specific [Versioning](/README.md).

## [Unreleased]

### Changed

- Updated SonarAnalyzer.CSharp to version 9.32.0.97167

### Added

- S2139: Exceptions should be either logged or rethrown but not both
- S2629: Logging templates should be constant
- S3431: "[ExpectedException]" should not be used
- S4347: Secure random number generators should not output predictable values
- S5344: Passwords should not be stored in plaintext or with a fast hashing algorithm
- S6377: XML signatures should be validated securely
- S6667: Logging in a catch clause should pass the caught exception as a parameter.
- S6668: Logging arguments should be passed to the correct parameter
- S6670: "Trace.Write" and "Trace.WriteLine" should not be used
- S6672: Generic logger injection should match enclosing type
- S6673: Log message template placeholders should be in the right order
- S6674: Log message template should be syntactically correct
- S6675: "Trace.WriteLineIf" should not be used with "TraceSwitch" levels
- S6677: Message template placeholders should be unique
- S6678: Use PascalCase for named placeholders
- S6781: JWT secret keys should not be disclosed
- S6930: Backslash should be avoided in route templates
- S6931: ASP.NET controller actions should not have a route template starting with "/"
- S6932: Use model binding instead of reading raw request data
- S6934: A Route attribute should be added to the controller when a route template is specified at the action level
- S6960: Controllers should not have mixed responsibilities
- S6961: API Controllers should derive from ControllerBase instead of Controller
- S6962: You should pool HTTP connections with HttpClientFactory
- S6964: Value type property used as input in a controller action should be nullable, required or annotated with the JsonRequiredAttribute to avoid under-posting.
- S6965: REST API actions should be annotated with an HTTP verb attribute
- S6966: Awaitable method should be used
- S6967: ModelState.IsValid should be called in controller actions
- S6968: Actions that return a value should be annotated with ProducesResponseTypeAttribute containing the return type

### Removed

- S6803: Parameters with SupplyParameterFromQuery attribute should be used only in routable components

## [3.2.1] - 2024-11-18

### Fixed
Expand Down
2 changes: 1 addition & 1 deletion Neolution.CodeAnalysis.TestsRuleset.nuspec
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
<developmentDependency>true</developmentDependency>
<dependencies>
<dependency id="StyleCop.Analyzers.Unstable" version="1.2.0.556" />
<dependency id="SonarAnalyzer.CSharp" version="9.20.0.85982" />
<dependency id="SonarAnalyzer.CSharp" version="9.32.0.97167" />
</dependencies>
</metadata>
<files>
Expand Down
2 changes: 1 addition & 1 deletion Neolution.CodeAnalysis.nuspec
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
<developmentDependency>true</developmentDependency>
<dependencies>
<dependency id="StyleCop.Analyzers.Unstable" version="1.2.0.556" />
<dependency id="SonarAnalyzer.CSharp" version="9.20.0.85982" />
<dependency id="SonarAnalyzer.CSharp" version="9.32.0.97167" />
</dependencies>
</metadata>
<files>
Expand Down
Loading