-
Notifications
You must be signed in to change notification settings - Fork 22
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
Support for Custom HTTP Headers / Mixed Mode Authentication #32
Comments
Just linking these issues together koltyakov/sppull#20 |
Hi Chris, thank you, To be honest I don't want to add any additional http-related settings to spsave, because from my POV it violates some design principles. However I need to provide a way for you to configure underlying So my proposal to fix your issue is using nodejs process environment variables.
You in your vscode add-in need to add below line before using
This approach also solves Some notes regarding basic authentication. Currently |
My opinion, that it's a bright idea! I also can add merging these custom headers in the part of SPPull, which doesn't use sp-request (there is one case with downloading large files as a stream). |
So I've updated process.env['_sp_request_headers'] = JSON.stringify({
'X-FORMS_BASED_AUTH_ACCEPTED': 'f'
}); |
Hello @s-KaiNet!
First: Thank you for authoring spsave! I am using this library as part of my VSCode addon: SPGo (https://github.com/readysitego/spgo) and it has saved me hours of work.
A number of my users support SharePoint environments where mixed mode authentication is used, but are unable to authenticate using NTLM Credentials due to the requirement that the "X-FORMS_BASED_AUTH_ACCEPTED” header be present, with a value of "f" (this configuration forces SharePoint to use NTLM auth).
Can you extend the spsave() function to also optionally accept a set of headers to include in all http/s calls to the SharePoint server?
Thanks!
Chris
The text was updated successfully, but these errors were encountered: