Skip to content

Update OpenApi implementation to avoid reflection - #1185

Closed
Youssef1313 wants to merge 4 commits into
dotnet:mainfrom
Youssef1313:dev/ygerges/no-refl
Closed

Update OpenApi implementation to avoid reflection#1185
Youssef1313 wants to merge 4 commits into
dotnet:mainfrom
Youssef1313:dev/ygerges/no-refl

Fix warnings that are now occurring

b492d06
Select commit
Loading
Failed to load commit list.
GitHub Advanced Security / CodeQL succeeded May 13, 2026 in 3s

3 new alerts

New alerts in code changed by this pull request

  • 2 warnings
  • 1 note

See annotations below for details.

View all branch alerts.

Annotations

Check warning on line 54 in src/AspNetCore/WebApi/src/Asp.Versioning.OpenApi/DependencyInjection/AggregateKeyedServiceProvider.cs

See this annotation in the file changed.

Code scanning / CodeQL

Constant condition Warning

Condition is always false because of
access to field initialized
Loading
.

Check warning on line 94 in src/AspNetCore/WebApi/src/Asp.Versioning.OpenApi/DependencyInjection/IApiVersioningBuilderExtensions.cs

See this annotation in the file changed.

Code scanning / CodeQL

Erroneous class compare Warning

Erroneous class compare.

Check notice on line 98 in src/AspNetCore/WebApi/src/Asp.Versioning.OpenApi/DependencyInjection/IApiVersioningBuilderExtensions.cs

See this annotation in the file changed.

Code scanning / CodeQL

Missed opportunity to use Where Note

This foreach loop
implicitly filters its target sequence
Loading
- consider filtering the sequence explicitly using '.Where(...)'.