question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

[Bug] Public cloud regionalization breaks existing services before 4.49.x

See original GitHub issue

Logs and network traces Without logs or traces, it is unlikely that the team can investigate your issue. Capturing logs and network traces is described in Logging wiki.

Which version of MSAL.NET are you using?

4.49.1

Platform

What authentication flow has the issue?

  • Desktop / Mobile
    • Interactive
    • Integrated Windows Authentication
    • Username Password
    • Device code flow (browserless)
  • Web app
    • Authorization code
    • On-Behalf-Of
  • Daemon app
    • Service to Service calls

Other?

Relates to https://github.com/AzureAD/microsoft-authentication-library-for-dotnet/issues/3252

Is this a new or existing app?

The 1P service is in production, and we have upgraded to a 4.49.1 or above of MSAL. This is breaking customers which use VNETs as they have to allow the traffic.

Repro

builder.WithAzureRegion(region);

Expected behavior

Not break customers.

Actual behavior

Customers have to change network to allow traffic to be sent to new regional endpoints.

Possible solution

Make the DNS suffix configurable so that 1P services can use later version of the package; but still use the old DNS suffix.

For example: builder.WithAzureRegion(region, regionalHostSuffix: "r.login.microsoftonline.com")

Issue Analytics

  • State:closed
  • Created 6 months ago
  • Comments:9 (6 by maintainers)

github_iconTop GitHub Comments

1reaction
bgavrilMScommented, Mar 30, 2023

After discussions with the ESTS-R team, we have reched the conclusion that we cannot enforce the new alias, as some end-app devs block it and it is outside our control.

We need a mechanism to:

  • perform the regional call on <region>.microsoft.login.com (like today)
  • catch exception and log
  • retry with <region>.r.login.microsoftonline.com

I propose we use either HttpClient extensibility or OnBeforeTokenRequest to achieve this. https://github.com/AzureAD/microsoft-authentication-library-for-dotnet/blob/main/src/client/Microsoft.Identity.Client/Extensibility/AbstractConfidentialClientAcquireTokenParameterBuilderExtension.cs#L23

MSAL team to provide a POC.

0reactions
bgavrilMScommented, Apr 6, 2023

Closing this as resolved, please re-open if more guidance is needed.

Read more comments on GitHub >

github_iconTop Results From Across the Web

Global warming of 1.5°C
Please use the following reference to the whole report: IPCC, 2018: Global Warming of 1.5°C. An IPCC Special Report on the impacts of...
Read more >
Collaborative Approach to Economic Development of Local ...
This dissertation investigates two types of cooperative relationships in the economic development arena that have not been given adequate attention and can not ......
Read more >
Agenda - Utah DEQ Document Repository
A. July 18, 2014 Board Meeting. •. Tage Flint moved to approve the minutes. Roger Fridal seconded. The motion was.
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found