Skip to content

Upgrade Microsoft.Extensions.Diagnostics.HealthChecks to 8.0.6#2252

Closed
Alirexaa wants to merge 2 commits intoXabaril:masterfrom
Alirexaa:fix/#2250
Closed

Upgrade Microsoft.Extensions.Diagnostics.HealthChecks to 8.0.6#2252
Alirexaa wants to merge 2 commits intoXabaril:masterfrom
Alirexaa:fix/#2250

Conversation

@Alirexaa
Copy link
Copy Markdown
Collaborator

What this PR does / why we need it:

Which issue(s) this PR fixes:
CVE-2024-21386
Please reference the issue this PR will close: #2250

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Unit tests passing
  • End-to-end tests passing
  • Extended the documentation
  • Provided sample for the feature

Copy link
Copy Markdown
Collaborator

@adamsitnik adamsitnik left a comment

Choose a reason for hiding this comment

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

Please wait until #2250 (comment) gets clarified.

@Alirexaa thank you for your help!


<ItemGroup>
<PackageReference Include="Microsoft.Extensions.Diagnostics.HealthChecks" Version="8.0.0" />
<PackageReference Include="Microsoft.Extensions.Diagnostics.HealthChecks" Version="8.0.6" />
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

It seems that all projects reference it, so we should most likely move this reference to the Directory.Build.props file here:

<ItemGroup>
<PackageReference Include="Roslynator.Analyzers" Version="4.7.0">

So next time we update only one file.

@adamsitnik
Copy link
Copy Markdown
Collaborator

Please see #2250 (comment) for details

@adamsitnik adamsitnik closed this Jun 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment