Skip to content
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

Deprecate Authentication=ActiveDirectoryPassword #2623

Open
David-Engel opened this issue Feb 28, 2025 · 0 comments · May be fixed by #2624
Open

Deprecate Authentication=ActiveDirectoryPassword #2623

David-Engel opened this issue Feb 28, 2025 · 0 comments · May be fixed by #2624
Assignees
Labels
Under Review Used for pull requests under review

Comments

@David-Engel
Copy link
Collaborator

Given the direction of Azure authentication and enforcement of mandatory MFA, we should deprecate the ActiveDirectoryPassword Authentication option on SqlConnections. MSAL and Azure.Identity are also planning to deprecate their corresponding APIs soon.

For more details of the Azure plan, see Planning for mandatory multifactor authentication for Azure and other admin portals

I don't think we have any APIs we need to annotate (unfortunately - would be nice to be able to provide compile time warnings). Otherwise, this just means updating the public docs, and including it in the next release announcement.

We should consider logging an INFO or WARN message if the option is used as we don't have other good signals to users to stop using the option.

Let me know if I'm missing anything else.

@github-project-automation github-project-automation bot moved this to To be triaged in MSSQL JDBC Feb 28, 2025
@machavan machavan self-assigned this Mar 3, 2025
@machavan machavan added the Work in Progress The pull request is a work in progress label Mar 3, 2025
@machavan machavan added Under Review Used for pull requests under review and removed Work in Progress The pull request is a work in progress labels Mar 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Under Review Used for pull requests under review
Projects
Status: To be triaged
Development

Successfully merging a pull request may close this issue.

2 participants