Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implementation Based on Feature Request #7151

Merged
merged 6 commits into from
Jun 7, 2024

Conversation

Robbie-Microsoft
Copy link
Collaborator

No description provided.

@github-actions github-actions bot added the msal-node Related to msal-node package label Jun 7, 2024
lib/msal-node/src/utils/Constants.ts Outdated Show resolved Hide resolved
const secretFilePath = wwwAuthHeader.split("Basic realm=")[1];

// throw an error if the managed identity application is not being run on Windows or Linux
if (
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Strange spacing for paranthesis etc.

Copy link
Collaborator Author

@Robbie-Microsoft Robbie-Microsoft Jun 7, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's done automatically by the "prettier" package we use to auto-format code.

@Robbie-Microsoft Robbie-Microsoft merged commit 3ee9c68 into dev Jun 7, 2024
9 checks passed
@Robbie-Microsoft Robbie-Microsoft deleted the implementation_for_feature_request branch June 7, 2024 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
msal-node Related to msal-node package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants