Skip to content
This repository was archived by the owner on Jan 5, 2026. It is now read-only.

fix: Remove undefined value in @azure/msal-node - #4718

Merged
tracyboehrer (tracyboehrer) merged 2 commits into
mainfrom
southworks/fix/remove-undefined-confidentialclientapplication
Jul 26, 2024
Merged

tracyboehrer (tracyboehrer) merged 2 commits into
mainfrom
southworks/fix/remove-undefined-confidentialclientapplication

Conversation

@JhontSouth

Copy link
Copy Markdown
Contributor

#minor

Description

This PR avoids the undefined reference for thumbprint value in the ConfidentialClientApplication instance in msalAppCredentials for the latest versions of @azure/msal-node.

Specific Changes

  • Replaced undefined value by an empty string in clientSecret definition.

Testing

The following image shows a bot working with node 20 and the latest version of @azure/msal-node after the update.
image

@coveralls

Coveralls (coveralls) commented Jul 24, 2024

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 10080470805

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.4%) to 84.433%

Totals Coverage Status
Change from base Build 9861924905: 0.4%
Covered Lines: 20429
Relevant Lines: 22904

💛 - Coveralls

@tracyboehrer
tracyboehrer (tracyboehrer) deleted the southworks/fix/remove-undefined-confidentialclientapplication branch July 26, 2024 14:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants