Skip to content

making the proxy with NTLM work (for grizzly provider) #266

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

Closed
wants to merge 4 commits into from

Conversation

vedil
Copy link
Contributor

@vedil vedil commented Apr 2, 2013

handling the proxy's NTLM auth, using NTLMEngine to generate required tokens.
detecting first, second level of handshakes and generating appropriate message types

@vedil
Copy link
Contributor Author

vedil commented Apr 3, 2013

basically this pull request is sum of two check-ins first one was working cut the second one is removing some not necessary code.

@rlubke
Copy link
Contributor

rlubke commented Apr 3, 2013

For whatever reason, the pull request cannot be merged. Would you mind sending the complete diff to [email protected]?

@vedil
Copy link
Contributor Author

vedil commented Apr 4, 2013

I have committed kerberos related changes too to my branch and this pull request does not contain it.
could that be a problem?
shall i send you a zip of my source (that includes kerberos changes)?

@vedil
Copy link
Contributor Author

vedil commented Apr 4, 2013

would it help if i close this and make another pull request which contains both ntlm and kerberos related changes?

@rlubke
Copy link
Contributor

rlubke commented Apr 4, 2013

Sure, let's give it a shot.

@vedil vedil closed this Apr 4, 2013
@vedil
Copy link
Contributor Author

vedil commented Apr 4, 2013

closed this and opened created another request, this contains kerberos related changes as well.

cs-workco pushed a commit to cs-workco/async-http-client that referenced this pull request Apr 13, 2023
`redirectConfiguration` can't default to `false` as it's not a boolean value, and the default value is `RedirectConfiguration()`.

Co-authored-by: Johannes Weiss <[email protected]>
Co-authored-by: Artem Redkin <[email protected]>
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