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

NTLM support #405

Closed
wants to merge 1 commit into from
Closed

NTLM support #405

wants to merge 1 commit into from

Conversation

jayzes
Copy link

@jayzes jayzes commented Mar 7, 2013

savonrb/httpi#80 brings in up-to-date NTLM support for HTTPI, so I figured I'd take a stab at putting in the appropriate hooks to do NTLM auth in a Savon client too.

This probably needs to wait for a new version of HTTPI to be released, however, so the dependency version can be bumped too. Happy to make that change and add it to this PR when the new version is out.

@rubiii
Copy link
Contributor

rubiii commented Mar 7, 2013

thanks. i need to come up with a release plan for new httpi features in master together with @rogerleite.

@rogerleite
Copy link
Member

Nice work! Let's work @rubiii !!! 👷

@rubiii
Copy link
Contributor

rubiii commented Mar 7, 2013

@rogerleite i'm hanging around in #httpirb and #savonrb on freenode (just created those) if you have a few minutes.

@Wardrop
Copy link

Wardrop commented May 1, 2013

Any update on this. I need to use Savon with SharePoint without screwing around with the configured server-side authentication mechanisms. In other words, I really need NTLM support. By the way, if you do include support, pelase make sure NTLM can be used to retrieve the WSDL as well.

@jayzes
Copy link
Author

jayzes commented May 1, 2013

Looks like HTTPI is still awaiting the next version release for the underlying NTLM support that this patch makes use of.

@rubiii
Copy link
Contributor

rubiii commented May 5, 2013

@Wardrop i'm not sure that's covered right now (using ntlm for retrieving the wsdl).

@Wardrop
Copy link

Wardrop commented May 5, 2013

@rubii It seems to be working with basic authentication. I know that Savon 1.x didn't use to work like this.

@rubiii rubiii mentioned this pull request Jul 24, 2013
27 tasks
@rubiii
Copy link
Contributor

rubiii commented Jul 26, 2013

merged into the version2 branch.

@rubiii rubiii closed this Jul 26, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants