Skip to content

Enable the new Broker discovery on MSAL side by default, Fixes AB#3226173#2618

Merged
rpdome merged 2 commits intodevfrom
rapong/enableDiscoveryByDefault
Apr 4, 2025
Merged

Enable the new Broker discovery on MSAL side by default, Fixes AB#3226173#2618
rpdome merged 2 commits intodevfrom
rapong/enableDiscoveryByDefault

Conversation

@rpdome
Copy link
Copy Markdown
Member

@rpdome rpdome commented Apr 2, 2025

Keeps the method to avoid introducing breaking change to OneAuth.

AB#3226173

@rpdome rpdome requested a review from a team as a code owner April 2, 2025 21:27
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 2, 2025

❌ Work item link check failed. Description does not contain AB#{ID}.

Click here to Learn more.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 2, 2025

✅ Work item link check complete. Description contains link AB#3226173 to an Azure Boards work item.

@github-actions github-actions Bot changed the title Enable Broker discovery on MSAL/OneAuth side by default Enable Broker discovery on MSAL/OneAuth side by default, Fixes AB#3226173 Apr 2, 2025
@rpdome rpdome changed the title Enable Broker discovery on MSAL/OneAuth side by default, Fixes AB#3226173 Enable Broker discovery on MSAL side by default, Fixes AB#3226173 Apr 2, 2025
@rpdome rpdome changed the title Enable Broker discovery on MSAL side by default, Fixes AB#3226173 Enable the new Broker discovery on MSAL side by default, Fixes AB#3226173 Apr 2, 2025
@mohitc1
Copy link
Copy Markdown
Contributor

mohitc1 commented Apr 3, 2025

Will there be any impact on E2E tests?

@rpdome rpdome merged commit ab89925 into dev Apr 4, 2025
22 of 25 checks passed
rpdome added a commit that referenced this pull request Apr 14, 2025
…s AB#3226173 (#2618)"

This reverts commit ab89925.

# Conflicts:
#	changelog.txt
rpdome added a commit that referenced this pull request Apr 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants