-
Notifications
You must be signed in to change notification settings - Fork 428
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
Updated MSAL and Azure Identity dependency versions #2584
base: main
Are you sure you want to change the base?
Conversation
350c92c
to
f9c184b
Compare
<azure-identity.version>1.12.2</azure-identity.version> | ||
<msal.version>1.15.1</msal.version> | ||
<azure-identity.version>1.15.0</azure-identity.version> | ||
<msal.version>1.17.2</msal.version> |
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.
why 1.17.2? latest is 1.18.0
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.
Dependency version conflicts, latest azure-identity 1.15.0 depends on msal 1.17.2. If I use 1.18.0 it breaks things.
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.
there are also later versions of antlr-runtime, com.google.code.gson, bcprov-jdk18on, bcpkix-jdk18on, junit-jupiter, hikaricp, gremini.mock, h2
please check all dependencies and update to latest stable release versions
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2584 +/- ##
============================================
+ Coverage 51.01% 51.17% +0.15%
- Complexity 3921 3942 +21
============================================
Files 147 147
Lines 33483 33505 +22
Branches 5609 5614 +5
============================================
+ Hits 17081 17145 +64
+ Misses 13991 13959 -32
+ Partials 2411 2401 -10 ☔ View full report in Codecov by Sentry. |
Updated the pom.xml with the latest versions of MSAL and Azure Identity.
The project's public Azure Artifact feed is also updated with the latest versions of MSAL and Azure Identity.
To test locally, pull this branch and do a fresh local build with
mvn clean install -DskipTests -T 1C
. If it works, your local build run should complete successfully without errors.Search for
Update Public Azure Artifacts Feed
in the internal driver docs (not public) to learn more.