-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126991/
-----------------------------------------------------------

(Updated Feb. 9, 2016, 10:53 p.m.)


Review request for KDE Frameworks and Dawit Alemayehu.


Changes
-------

Addressed David's comments.


Repository: kio


Description
-------

When authenticating agains a server offering multiple authentication methods 
make sure to attempt other methods in case the best one fails.
    
This also fixes a connection close issue in the middle of an NTLM 
authentication dialog due to clearing the password.


Diffs (updated)
-----

  src/ioslaves/http/http.h 621b2c7a957b9bc9cc14ff13ed3c3a72dec38190 
  src/ioslaves/http/http.cpp e1013c8705e6588729d61ed45c43dc564415c41e 

Diff: https://git.reviewboard.kde.org/r/126991/diff/


Testing
-------

I have performed testing on an IIS 7.5 server which offered 3 authentication 
options: Negotiate, NTLM and Basic. Since I have Kerberos configured the 
original code would only try Negotiate and because it failed it would retry it 
endlessly. With this patch authentication correctly falls back to NTLM or Basic 
(if NTLM fails too).


Thanks,

Krzysztof Nowicki

_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Reply via email to