search

colynb / request-ntlm

NTLM lib for nodejs
5 stars 24 forks source link

two small-ish fixes #2

Open FrankyBoy opened 9 years ago

FrankyBoy commented 9 years ago

for #1 and #3 (tbd)

colynb commented 9 years ago

Thanks for this @FrankyBoy - A large team is currently using this for a large build process that uses this for connecting to an older api (which is why I built this) so I'll need to do some testing for a bit before I can get this merged. But thanks for your contribution!

FrankyBoy commented 9 years ago

Jeah, totally. I am just currently writing node bindings for some SOAP-services that use ntlm (the joy ... both to soap and ntlm) so I'm just patching in stuff as I go along to suit my needs while trying not to break anything major ;)