search

Azure / azure-sdk-for-js

This repository is for active development of the Azure SDK for JavaScript (NodeJS & Browser). For consumers of the SDK we recommend visiting our public developer docs at https://docs.microsoft.com/javascript/azure/ or our versioned developer docs at https://azure.github.io/azure-sdk-for-js.
MIT License
1.98k stars 1.15k forks source link

[identity] Delete MsalFlow code #29900

Open maorleger opened 4 weeks ago

maorleger commented 4 weeks ago

Once all the Node credentials have migrated over and we released a GA version with all the credentials migrated over we can finally delete the node MSALFlow code

That may include:

This does not include:

I recommend then creating issues to migrate the remaining browserFlows to a more sane pattern, but it is out of scope for this project

maorleger commented 4 weeks ago

The reason we want to GA a version with the existing code in place is so we can hotfix easily in case of any issues.

A hotfix will simply swap out the msalClient for MsalNode` flows in the credential but if we GA a version that no longer has the MsalNode code in place it will be much harder to hotfix