Skip to content

Conversation

@steventebrinke
Copy link

Updates the OData Service to allow passing it a block that is called when Faraday is initialised. This allows more advanced configurations such as NTLM auth, of which I give an example in the README. (Since NTLM auth with Faraday is undocumented, my way of using it might not be the best. Therefore, this code is kept outside the OData library, so other ways of reconfiguring Faraday are also possible.)

@visoft
Copy link
Owner

visoft commented Aug 8, 2017

I'm so sorry that I missed this pull request. Great job! The one thing I'd like to see is some specs testing out NTLM auth. I love the README addition, that will help a lot of users. I plan on rolling out a new version and would like to add this. Do you have time to add some specs?

@visoft
Copy link
Owner

visoft commented Aug 8, 2017

I pulled your changes in the develop branch, but a bunch of specs failed. I didn't check anything into GitHub. Could you revisit the changes against the develop branch?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants