-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
PoP Support for Node when brokered #7360
Conversation
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.
Nothing we need to commit in this PR but some future considerations.
@@ -1030,6 +1161,7 @@ if (process.platform === "win32") { | |||
const result: AuthResult = { | |||
idToken: "", | |||
accessToken: "", | |||
authorizationHeader: "", |
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.
Tons of duplicate code in this PR, can we do anything to eliminate the duplication?
| Name | Description | Required | | ||
|-------------------------| ----------------------------------------------------------- | -------- | | ||
| `authenticationScheme` | Indicates whether MSAL should acquire a `Bearer` or `PoP` token. Default is `Bearer`. | **Required** | | ||
| `resourceRequestMethod` | The all-caps name of the HTTP method of the request that will use the signed token (`GET`, `POST`, `PUT`, etc.) | **Required** | |
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.
I'd generally suggest we uppercase the incoming value rather than asking for all-caps input.
…reAD/microsoft-authentication-library-for-js into pop-support-in-node-brokered
…tion-library-for-js into pop-support-in-node-brokered
…reAD/microsoft-authentication-library-for-js into pop-support-in-node-brokered
Fixes PoP support for Node when using the native broker