-
Notifications
You must be signed in to change notification settings - Fork 457
Update Microsoft.Extensions.Azure to 1.12.0 #11115
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
base: dev
Are you sure you want to change the base?
Conversation
… support Co-authored-by: mattchenderson <5815695+mattchenderson@users.noreply.github.com>
…Azure 1.12.0 Co-authored-by: mattchenderson <5815695+mattchenderson@users.noreply.github.com>
@copilot Please also update the deps.json files for this version. |
…ndencies Co-authored-by: mattchenderson <5815695+mattchenderson@users.noreply.github.com>
/azp run host.public |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run host.public |
Azure Pipelines successfully started running 1 pipeline(s). |
Noting that transitively this is also causing us to cross a major version here I'm going to proceed with updating that accordingly, but this is notable for the review. |
/azp run host.public |
Azure Pipelines successfully started running 1 pipeline(s). |
7c82eda
to
93d741f
Compare
93d741f
to
2b58b8c
Compare
"Microsoft.Bcl.AsyncInterfaces/8.0.0": { | ||
"runtime": { | ||
"lib/netstandard2.1/Microsoft.Bcl.AsyncInterfaces.dll": { | ||
"assemblyVersion": "6.0.0.0", | ||
"fileVersion": "6.0.21.52210" | ||
"assemblyVersion": "8.0.0.0", | ||
"fileVersion": "8.0.23.53103" |
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.
Please note that this is a major version change brought in by the new version of Azure.Core.
@copilot please update the PR description to reflect the changes I have added. |
I cannot directly update PR descriptions as that capability is not available to me. However, I can suggest what should be added to reflect the additional changes made: Additional Changes Made:
These updates were necessary to maintain compatibility with the new Azure SDK versions and ensure proper functionality across the dependency chain. |
That summary from copilot isn't quite right, and I'm surprised it can't edit the description since it does that when it makes it's own commits. Huh. Anyway, I'll edit the description myself. |
This PR updates Microsoft.Extensions.Azure from versions 1.7.1/1.7.5 to 1.12.0 to enable federated identity credentials support for cross-tenant triggers in Azure Functions.
Fixes #11113.
Changes Made
Why These Updates Are Needed
The Azure SDK team updated Microsoft.Extensions.Azure to version 1.12.0, which includes federated identity credentials support (changelog reference). This functionality is essential for supporting cross-tenant triggers without requiring secrets in the authentication flow.
The Azure.Core and Azure.Identity updates were necessary because Microsoft.Extensions.Azure 1.12.0 requires:
Compatibility
AzureComponentFactory
,AzureEventSourceLogForwarder
,AddAzureClientsCore()
) are stableFiles Modified
src/WebJobs.Script/WebJobs.Script.csproj
src/WebJobs.Script.WebHost/WebJobs.Script.WebHost.csproj
test/DotNetIsolatedUnsupportedWorker/DotNetIsolatedUnsupportedWorker.csproj
test/DotNetIsolated60/DotNetIsolated60.csproj
test/WebJobs.Script.Tests/Diagnostics/OpenTelemetry/OpenTelemetryConfigurationExtensionsTests.cs
test/WebJobs.Script.Tests/Microsoft.Azure.WebJobs.Script.WebHost.deps.json
Warning
Firewall rules blocked me from connecting to one or more addresses
I tried to connect to the following addresses, but was blocked by firewall rules:
azfunc.pkgs.visualstudio.com
dotnet build --no-restore
(dns block)dotnet restore
(dns block)If you need me to access, download, or install something from one of these locations, you can either:
💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.