-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
fix: Update Azure identity services (1.1.1 -> 1.11.1) and pygments (2.15.0 -> 2.15.1) #19576
Conversation
❌ Preview Environment deleted from BunnyshellAvailable commands (reply to this comment):
|
❌ Preview Environment deleted from BunnyshellAvailable commands (reply to this comment):
|
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.
Looks like tests are failing on change. These are pretty big version changes. Is there a version change that's closer and closes cves?
https://osv.dev/vulnerability/GHSA-m5vv-6r4h-3vj9 states that for the Go Azure SDK, the flaw was fixed in v1.6.0, while this PR pulls in 1.11 (which is probably the latest stable release). However, I think it might be worthwhile to fix the compatibility issue. EDIT: It seems the vulnerability is in the |
Better has an Azure environment to do a validation. |
Signed-off-by: harshitasao <[email protected]>
Signed-off-by: harshitasao <[email protected]>
b8f51e6
to
9f4b57a
Compare
We got pyments done in #19784 - this azure change is significant though and needs real testing. |
This has been solved with #20578. |
Fixed the 2 vulnerabilities which are reported by the scorecard report.
Part of #18921
Checklist: