Review Request: Fix kio_http for http requests to locations requiring credentials that return no body

Matthew Dawson matthewjd at gmail.com
Sat Sep 19 16:43:36 BST 2009


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1236/
-----------------------------------------------------------

(Updated 2009-09-19 15:43:36.478538)


Review request for kdelibs.


Changes
-------

Update patch to what is to be committed.


Summary
-------

This patch fixes commit r707403.  Commit r707403 makes websites that require http authentication , but return no error page, kio_http returns an error of page not found.  Commit r707403 attempts to fix errors in Konqueror when no error page is returned by the server when an error occurs.  It also blocks a 401 from being properly handled when no error page is returned.  This bug is set off when eGroupware is used with the groupdav resource (thus the related bug).  This patch fixes the issue by ignoring a 401 error code in the handling.  401's do not cause the original issue that Commit r707403 fixed.


This addresses bug 186763.
    https://bugs.kde.org/show_bug.cgi?id=186763


Diffs (updated)
-----

  /trunk/KDE/kdelibs/kioslave/http/http.cpp 1014944 

Diff: http://reviewboard.kde.org/r/1236/diff


Testing
-------

Removing the 401 from the case doesn't cause the original bug report (147812) to regress.  When clicking cancel on the password dialog, the same error page occurs in konqueror.  When clicking ok with a blank username/password, nothing happens, but the same behaviour happens without the patch.  Wrong passwords do the same thing as well.

Testing done with KDE 4.3.0 and KDE svn.  Web server was lighttpd.  Authentication was tested with mod_auth and the groupdav support in egroupware running on the local system.  Tested with both kioclient and konqueror.


Thanks,

Matthew





More information about the kde-core-devel mailing list