EWS: Support NTLM authentication

mustang-im / mustang

Mustang - New full-featured desktop email, chat and video conference client

https://mustang.im

Other

5 stars 0 forks source link

Open benbucksch opened 2 weeks ago

benbucksch commented 2 weeks ago

Check how difficult it is. If easy (< 4 hours), please implement it.

At the very least, don't show "Password incorrect", but "NTLM not supported".

NeilRashbrook commented 1 week ago

Apparently this used to be easier back in Node 16 which still included the DES and MD4 code.

There's an httpntlm npm library; this normally wants to do its own HTTP requests but you can call the NTLM header handling functions manually.
There's an @ewsjs/ntlm-client library. This just seems to provide the NTLM header handling functions, which might explain why it's relatively unpopular. It's only half the size though.

benbucksch commented 1 week ago

(Smaller is better. Less dependencies is better.)

Can you quickly check the code quality of the @ewsjs/ntlm-client lib?

NeilRashbrook commented 1 week ago

npmjs.com rates it as 53% as compared to httpntlm's 75%, probably because it has no tests included. The code itself looks reasonably clean to me.

benbucksch commented 1 week ago

I trust your judgment far more than some algo :-) . OK, go for it.