Skip to content
This repository was archived by the owner on Jan 5, 2026. It is now read-only.

Conversation

@boydc2014
Copy link
Contributor

@boydc2014 boydc2014 commented Mar 25, 2021

Refs dotnet/runtime#49377 (comment)

Fixes #5356

Description

System.Text.Encodings.Web has security issues needs to be fixed.

Most of our libraries don't directly depend on this particular package, in most cases, we depend on it via asp.net core. And after .net core 3.0, the asp.net core is built-into .net core runtime, so usually user just have to update the .net core runtime in their end, we should be OK.

One exception is in this particular package, we depend on "Microsoft.ApplicationInsights.AspNetCore" and then later depend on this offending package via package reference, which can't be solved by updating runtime.

So, in this fix, i override the transient dependency directly.

See also microsoft/BotFramework-Composer#6548

After the fix
image

@boydc2014 boydc2014 requested a review from a team as a code owner March 25, 2021 05:07
@mrivera-ms mrivera-ms self-requested a review March 25, 2021 18:41
@mrivera-ms mrivera-ms merged commit de27fc0 into main Mar 25, 2021
@mrivera-ms mrivera-ms deleted the dong/fix-security branch March 25, 2021 18:43
@boydc2014 boydc2014 restored the dong/fix-security branch April 22, 2021 02:41
@EricDahlvang EricDahlvang deleted the dong/fix-security branch September 22, 2021 15:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Microsoft Security Advisory CVE-2021-26701

3 participants