Commit 10395be
authored
[Identity] Making MSAL extensions an unlisted dependency (Azure#14418)
`@azure/msal-node-extensions` has some very inconvenient requirements for our developers and for our community. It requires (besides Python in the three environments):
- On Linux, to have installed `libsecret` and `build-essentials`.
- On OSX, to have XCode configured.
- On Windows: To have Visual Studio Code installed with C++ (this is reportedly about 60gb)
While we move this dependency (and KeyTar) to a new package, this PR makes it so the dependency is "unlisted": If users try to use this feature, we will throw an error asking them to install the dependency. Tests that use this feature get skipped if the dependency is not present, and run if the dependency is present.
I will add more clarity in the docs as part of this other PR: Azure#144051 parent 5bd519c commit 10395be
File tree
20 files changed
+547
-511
lines changed- common/config/rush
- sdk/identity/identity
- recordings/node
- clientcertificatecredential_internal_persistent_tests
- clientsecretcredential_internal_persistent_tests
- devicecodecredential_internal_persistent_tests
- usernamepasswordcredential_internal_persistent_tests
- src/tokenCache
- test/internal/node
20 files changed
+547
-511
lines changedSome generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
87 | 87 | | |
88 | 88 | | |
89 | 89 | | |
90 | | - | |
91 | 90 | | |
92 | 91 | | |
93 | 92 | | |
| |||
0 commit comments