WorkloadIdentityCredential: TypeError: Cannot read property 'tenantId' of undefined

[brianpham93]

• Package Name: @azure/identity
• Package Version: 3.2.0
• Operating system:
• nodejs
  • version: 16

Describe the bug
If DefaultAzureCredential is used, this error will throw because there's no AZURE_TENANT_ID in environment variables

TypeError: Cannot read property 'tenantId' of undefined
    at new WorkloadIdentityCredential (<directory>\node_modules\@azure\identity\dist\index.js:1999:60)
    at new DefaultWorkloadIdentityCredential (<directory>\node_modules\@azure\identity\dist\index.js:3589:13)
    at <directory>\node_modules\@azure\identity\dist\index.js:3622:51
    at Array.map (<anonymous>)
    at new DefaultAzureCredential (<directory>\node_modules\@azure\identity\dist\index.js:3622:37)
    at Object.<anonymous> (<directory>\util\keyvault-service.js:1:2980)
    at Module._compile (internal/modules/cjs/loader.js:1085:14)
    at Module.replacementCompile (<directory>\node_modules\append-transform\index.js:58:13)
    at Module._extensions..js (internal/modules/cjs/loader.js:1114:10)
    at Object.<anonymous> (<directory>\node_modules\append-transform\index.js:62:4)
    at Module.load (internal/modules/cjs/loader.js:950:32)
    at Function.Module._load (internal/modules/cjs/loader.js:790:12)
    at Module.require (internal/modules/cjs/loader.js:974:19)
    at Module.patchedRequire [as require] (<directory>\node_modules\diagnostic-channel\dist\src\patchRequire.js:14:46)
    at require (internal/modules/cjs/helpers.js:93:18)
    at Object.<anonymous> (<directory>\util\key-management.js:1:1998)
    at Module._compile (internal/modules/cjs/loader.js:1085:14)
    at Module.replacementCompile (<directory>\node_modules\append-transform\index.js:58:13)
    at Module._extensions..js (internal/modules/cjs/loader.js:1114:10)
    at Object.<anonymous> (<directory>\node_modules\append-transform\index.js:62:4)
    at Module.load (internal/modules/cjs/loader.js:950:32)
    at Function.Module._load (internal/modules/cjs/loader.js:790:12)
    at Module.require (internal/modules/cjs/loader.js:974:19)
    at Module.patchedRequire [as require] (<directory>\node_modules\diagnostic-channel\dist\src\patchRequire.js:14:46)
    at require (internal/modules/cjs/helpers.js:93:18)
    at Object.<anonymous> (<directory>\util\secret-key-singleton.js:1:1964)
    at Module._compile (internal/modules/cjs/loader.js:1085:14)
    at Module.replacementCompile (<directory>\node_modules\append-transform\index.js:58:13)
    at Module._extensions..js (internal/modules/cjs/loader.js:1114:10)
    at Object.<anonymous> (<directory>\node_modules\append-transform\index.js:62:4)
    at Module.load (internal/modules/cjs/loader.js:950:32)

To Reproduce
Steps to reproduce the behavior:

1. Use DefaultAzureCredential

const { DefaultAzureCredential } = require("@azure/identity");
const defaultAzureCredential = new DefaultAzureCredential();

Expected behavior
It should fallback to next available authentication option

[orgads]

Proposed fix:

• [Identity] Fix undefined dereference #25829

[Apokalypt]

Same bug for us, waiting for the PR to be resolved

[MartaSleboda]

Same here, looking forward to fixing

[kanaga-divya]

Same here. Is this a new issue? I am seeing this from today morning.

[Apokalypt]

Same here. Is this a new issue? I am seeing this from today morning.

Yes, this was added with the last release which added default credentials when using the DefaultAzureCredential constructor. All the credentials methods handle the fact that the "options" parameter is not mandatory and can therefore be "undefined" except for one which does not control this and causes the instantiation of DefaultAzureCredential to crash.
You have 2 easy solutions (until the real solution is deployed):

• Rollback to @azure/identity v3.1.4
• Pass en empty parameter when instantiating DefaultAzureCredential : new DefaultAzureCredential({ })

[KarishmaGhiya]

The PR by @orgads has been merged. I am working on releasing the fix in version 3.2.1. Will update the thread when it's out.

[KarishmaGhiya]

Folks the version 3.2.1 for @azure/identity has released with the fix. Please feel free to use it. Also let me know if that works for you. Also thanks @orgads
cc: @Apokalypt @brianpham93 @MartaSleboda @kanaga-divya

[github-actions]

Hi @brianpham93. Thank you for opening this issue and giving us the opportunity to assist. To help our team better understand your issue and the details of your scenario please provide a response to the question asked above or the information requested above. This will help us more accurately address your issue.

[Apokalypt]

Folks the version 3.2.1 for @azure/identity has released with the fix. Please feel free to use it. Also let me know if that works for you. Also thanks @orgads cc: @Apokalypt @brianpham93 @MartaSleboda @kanaga-divya

On our side, the fix have corrected the problem.
Thx

[brianpham93]

Folks the version 3.2.1 for @azure/identity has released with the fix. Please feel free to use it. Also let me know if that works for you. Also thanks @orgads cc: @Apokalypt @brianpham93 @MartaSleboda @kanaga-divya

Thanks. The fix works for me

[KarishmaGhiya]

Thank you so much for the confirmation. I'll go ahead and mark this issue as resolved.