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

Add obsolete tag to MsalServiceClientCredentialsFactory class#6866

Merged
tracyboehrer merged 1 commit into
mainfrom
southworks/add/obsolete-tag-msalserviceclientcredentialsfactory
Nov 11, 2024
Merged

Add obsolete tag to MsalServiceClientCredentialsFactory class#6866
tracyboehrer merged 1 commit into
mainfrom
southworks/add/obsolete-tag-msalserviceclientcredentialsfactory

Conversation

@ceciliaavila

Copy link
Copy Markdown
Collaborator

#minor

Description

This PR adds the obsolete tag to the MsalServiceClientCredentialsFactory class, so users opt to use one of the other factory classes depending on the type of authentication they use (ConfigurationServiceClientCredentialFactory, CertificateServiceClientCredentialsFactory, FederatedServiceClientCredentialsFactory, etc).

Specific Changes

  • Update MsalServiceClientCredentialsFactory to add the Obsolete tag.

Testing

This image shows the obsolete tag added to the class.
image

@ceciliaavila ceciliaavila added the Automation: Parity with js The PR needs to be ported to JS label Nov 8, 2024
@ceciliaavila ceciliaavila requested a review from a team as a code owner November 8, 2024 13:42
@ceciliaavila

Copy link
Copy Markdown
Collaborator Author

Hi @tracyboehrer, this PR and a few others are failing with an error related to the private nuget feed. Let us know if there's anything we can do to help getting these merged.

@tracyboehrer tracyboehrer merged commit 913f7c3 into main Nov 11, 2024
@tracyboehrer tracyboehrer deleted the southworks/add/obsolete-tag-msalserviceclientcredentialsfactory branch November 11, 2024 16:40
This was referenced May 14, 2026
This was referenced May 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Automation: Parity with js The PR needs to be ported to JS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants