Bump SonarAnalyzer.CSharp from 9.24.0.89429 to 9.25.0.90414

[dependabot[bot]]

Bumps SonarAnalyzer.CSharp from 9.24.0.89429 to 9.25.0.90414.

Release notes

Sourced from SonarAnalyzer.CSharp's releases.

9.25

Hello everyone,

This release comes with seven new rules for ASP.NET core alongside some improvements. Enjoy!

New Rules

• 9096 - [C#] New Rule S6966: Awaitable method should be used
• 9095 - [C#] New Rule S6967: ModelState.IsValid should be called in controller actions
• 9094 - [C#] New Rule S6964: The value type properties of a model class should be nullable or marked as "Required" to avoid under-posting.
• 9093 - [C#] New rule S6968: Actions that return a value should be annotated with ProducesResponseTypeAttribute containing the return type
• 9092 - [C#] New rule S6965: You should use HttpAttribute in API controller actions
• 9091 - [C#] New rule S6962: You should pool HTTP connections with HttpClientFactory
• 9089 - [C#] New rule S6960: Controllers should not have too many responsibilities

Bug Fixes

• 9193 - Fix AD0001: Named Attribute Arguments in S6930

False Positive

• 9219 - [C#] Fix S1144 FP: Getters/Setters of property with attribute are being flagged

Improvements

• 9187 - Update RSPEC before 9.25 release
• 9186 - [C#] Rule S6961: Implement CodeFix

Rule deprecations and deletions

• 9175 - [VB.NET] Delete S2353: Remove deprecated rule
• 9189 - [C#] Deprecate S6803

Commits

• 75e3904 Update RSPEC before 9.25 release (#9246)
• bc4c63b Fix S6966 FPs/FNs: FPs/FNs after peach validation (#9222)
• e0dce12 New rule S6960 for C#: Controllers should not have too many responsibilities ...
• 7e0dada Fix Razor location UTs (#9242)
• 24ec493 Improve S6964: Include array types and nested collection types (#9239)
• c298484 Refactor: Extract common method to check if any ancestor belongs to SyntaxKin...
• ca0f118 Decompose SymbolHelper into dedicated extension classes (#9229)
• c755a17 Enable razor UTs in-memory compilation (#9226)
• 75e2d7b Repro for FN 2589: Support redundant recursive pattern null check (#9234)
• a2cd911 Fix S1144 FP: Getters/Setters of property with attribute are being flagged (#...
• Additional commits viewable in compare view

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

---

Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `@dependabot show ignore conditions` will show all of the ignore conditions of the specified dependency - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)