Skip to content
This repository was archived by the owner on Jan 5, 2026. It is now read-only.

Drop ADAL to unblock pipeline#6687

Merged
tracyboehrer merged 5 commits into
microsoft:mainfrom
Danieladu:hond/drop-adal
Sep 6, 2023
Merged

Drop ADAL to unblock pipeline#6687
tracyboehrer merged 5 commits into
microsoft:mainfrom
Danieladu:hond/drop-adal

Conversation

@Danieladu
Copy link
Copy Markdown
Contributor

@Danieladu Danieladu commented Aug 29, 2023

Fixes #minor

Description

ADAL library (Microsoft.IdentityModel.Clients.ActiveDirectory,Microsoft.Azure.Services.AppAuthentication ) has been marked deprecated and would block the Azure pipeline on 23 September.

Totally remove ADAL code to unblock the pipeline

@Danieladu
Copy link
Copy Markdown
Contributor Author

CI exception:
image
I think it make sense? Because we remove the ADAL code, which has been marked deprecate longlong ago.

@Danieladu Danieladu marked this pull request as ready for review August 30, 2023 10:00
@Danieladu Danieladu requested a review from a team as a code owner August 30, 2023 10:00
Copy link
Copy Markdown
Collaborator

@sw-joelmut sw-joelmut left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Danieladu,

The changes are looking good! About the CI exception, this PR is introducing breaking changes.

@tracyboehrer, to merge this PR you'll need to exclude these classes from the compat check by adding them to the list in the ApiCompatExcludeClasses variable.

For more information, check out the PR #5124.

@Danieladu
Copy link
Copy Markdown
Contributor Author

I have updated the variable, but seems not work
image
Is there any additional thing should i do?

@sw-joelmut
Copy link
Copy Markdown
Collaborator

Hi @Danieladu,

We want to share our findings regarding the ApiCompatExcludeClasses variable.
The variable is working as expected, but you will need to execute the pipeline (new run, not retry) for the changes to be applied.
imagen

Additionally, there is another way of excluding the classes.
All compatibility issues shown in the Compatibility Check pipeline's section can be excluded by adding them to the ApiCompatBaseline.txt file.
imagen

@Danieladu
Copy link
Copy Markdown
Contributor Author

Danieladu commented Sep 5, 2023

@microsoft/botframework-sdk @tracyboehrer please help review this PR which blocks the SDK pipeline and also ABS pipeline.

This was referenced May 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants