[cheenamalhotra]

Summary

• Mark azure-active-directory-password authentication as deprecated.
• Emit a one-time TEDIOUS_DEP_AAD_PASSWORD deprecation warning.
• Add regression coverage for warning behavior.

Fixes #1684

Validation

• npm test
• npx tsc --noEmit
• git diff --check

cc @arthurschreiber @saurabh500 @David-Engel

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.81%. Comparing base (fd1149d) to head (fba030a).

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1803      +/-   ##
==========================================
- Coverage   84.49%   83.81%   -0.68%     
==========================================
  Files          96       96              
  Lines        4720     4726       +6     
  Branches      884      885       +1     
==========================================
- Hits         3988     3961      -27     
- Misses        468      506      +38     
+ Partials      264      259       -5     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[saurabh500]

@arthurschreiber and @David-Engel can we take this? Such deprecation require heads up which other MS drivers have provided through documentation?

[arthurschreiber]

Isn't this feature mostly unusable anyway given the 2FA requirement that was added some while back?

[David-Engel]

Isn't this feature mostly unusable anyway given the 2FA requirement that was added some while back?

For personal Azure subscriptions, yes. For enterprise, it depends on the tenant configuration. Non-interactive AAD/Entra auth is a pain for us to even test these days due to policy restrictions but it's still a possible customer scenario.

[cheenamalhotra]

@arthurschreiber anything needed here to get this merged?