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
Msal Node adds support proxy
#4447
Conversation
Codecov Report
*This pull request uses carry forward flags. Click here to find out more.
|
…r sendPostRequestAsync for devidce flow.
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.
Looks good! Thanks for addressing feedback!
🎉 We recommend upgrading to the latest version of Handy links: |
🎉 Handy links: |
msal-node by default uses
axios
as the network manager.Since
axios
does not support proxies in its implementation as mentioned in this issue, the current SDK fails to support proxy environments.We provide couple options to solve this:
proxy
support explicitly with https-proxy-agentThis PR adds the
proxy
support in the existing package and adds documentation for alternatives.