Review Request: Fix auth warnings when authentication is legitimate
Grégory Oestreicher
greg at kamago.net
Thu Jun 2 13:03:58 BST 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101493/
-----------------------------------------------------------
Review request for kdelibs and Dawit Alemayehu.
Summary
-------
Commit 3bbd4496bc introduced spurious warnings when auth info was added to HTTP request and authentication succeeded as the response code will never be 401 or 407 in that case.
This patch takes into account the previous response code when deciding if a warning should be issued.
Diffs
-----
kioslave/http/http.cpp b4d3c65
Diff: http://git.reviewboard.kde.org/r/101493/diff
Testing
-------
Tested that the expected behaviour is still wanted, and that successful authentication attempts did not result in warnings.
Thanks,
Grégory
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20110602/26356bb2/attachment.htm>
More information about the kde-core-devel
mailing list