-
Notifications
You must be signed in to change notification settings - Fork 2.7k
[v5] Add PCAFactory and correlation id optional params to createNestablePublicClientApplication #8093
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
base: msal-v5
Are you sure you want to change the base?
Conversation
…blicClientApplication
…blicClientApplication
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR enhances the createNestablePublicClientApplication
function by adding two optional parameters: correlationId
and pcaFactory
. The enhancement allows users to provide their own correlation ID instead of auto-generating one and to use a custom factory function for creating the PublicClientApplication instance.
Key changes:
- Added optional
correlationId
parameter to allow custom correlation IDs - Added optional
pcaFactory
parameter to enable custom PCA instance creation - Updated function signature and implementation to handle the new parameters
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
File | Description |
---|---|
lib/msal-browser/src/app/PublicClientApplication.ts | Updated function signature and implementation to support optional correlationId and pcaFactory parameters |
lib/msal-browser/test/app/createNestablePublicClientApplication.spec.ts | Added comprehensive test coverage for the new optional parameters including edge cases |
lib/msal-browser/apiReview/msal-browser.api.md | Updated API documentation to reflect the new function signature |
change/@azure-msal-browser-e8a7eb79-1f90-43a2-8788-a2e901e835ca.json | Added beachball change file for the minor version update |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is an officeAddin sample in our repo, we may want to test it with the new PCA Params before merging this. Let me know if you need help there.
Add PCAFactory and correlation id optional params to createNestablePublicClientApplication