-
Notifications
You must be signed in to change notification settings - Fork 199
Upgrade version of Microsoft.Azure.WebJobs.Extensions.Http to remove vulnerability #493
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
Upgrade version of Microsoft.Azure.WebJobs.Extensions.Http to remove vulnerability #493
Conversation
…vulneratbility. There is a vulnerability in Microsoft.AspNetcore.Http in 2.1.0 that we need to upgrade References: - GHSA-hxrm-9w7p-39cc - dotnet/aspnetcore#24264
Hi @justinyoo - Please take a peek at this PR, there's a high vulnerability that should be resolved for consumers of this library. |
@justinyoo I will second the request on this item. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @jackbatzner for this PR. I've left some comments for you to take a look.
...Azure.WebJobs.Extensions.OpenApi.Core/Microsoft.Azure.WebJobs.Extensions.OpenApi.Core.csproj
Outdated
Show resolved
Hide resolved
...nApi.FunctionApp.InProc/Microsoft.Azure.WebJobs.Extensions.OpenApi.FunctionApp.InProc.csproj
Outdated
Show resolved
Hide resolved
...nApi.FunctionApp.InProc/Microsoft.Azure.WebJobs.Extensions.OpenApi.FunctionApp.InProc.csproj
Outdated
Show resolved
Hide resolved
...WebJobs.Extensions.OpenApi.TestApp/Microsoft.Azure.WebJobs.Extensions.OpenApi.TestApp.csproj
Outdated
Show resolved
Hide resolved
...WebJobs.Extensions.OpenApi.TestApp/Microsoft.Azure.WebJobs.Extensions.OpenApi.TestApp.csproj
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's focus on the NuGet package update in this PR. Please remove the two other C# file updates from this PR and create a new one.
src/Microsoft.Azure.WebJobs.Extensions.OpenApi.Core/Extensions/TypeExtensions.cs
Outdated
Show resolved
Hide resolved
...zure.WebJobs.Extensions.OpenApi.Core.Tests/Visitors/ListObjectInheritanceTypeVisitorTests.cs
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks for the PR
Thanks to @jackbatzner for the fix, and @justinyoo for being responsive! |
There is a vulnerability in Microsoft.AspNetcore.Http in 2.1.0 that we need to upgrade
References: