Remove pinned Azure packages #1185
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
We had pinned Azure.Core and Azure.Identity to resolve a CVE in the Azure.Identity package <1.11.4, but after looking into it that doesn't seem to be necessary since the version of SqlClient we use already requires that to be >=1.11.4
https://www.nuget.org/packages/Microsoft.Data.SqlClient/5.2.2#dependencies-body-tab
So having these direct dependencies is just adding unnecessary requirements on our package since we don't actually reference these directly.
Code Changes
az_func.GlobalState
table must be compatible with all prior versions of the extensionILogger
instance to log relevant information, especially information useful for debugging or troubleshootingasync
andawait
for all long-running operationsCancellationToken
Dependencies
dotnet restore --force-evaluate
to update the lock files and ensure that there are NO major versions updates in either src/packages.lock.json or Worker.Extensions.Sql/src/packages.lock.json. If there are, contact the dev team for instructions.Documentation