On Tuesday 07 January 2014 05:27:23 Steve Caturan wrote:
> hi all,
>
> I recently installed Konqueror on Windows 7 using KDE windows
> installer hoping to test drive it's WebDAV capabilities but came
> across specific issue I noted on the forum post at
> https://forum.kde.org/viewtopic.php?f=59&t=119075
>
> The issues are namely:
>
> - no support for TLS 1.1 and 1.2, only SSLv3 and TLS 1.0
>
> - missing slash when deleting remote directories (eg. DELETE
> /path/to/directory) to Apache/mod_dav renders a 301 redirect. Other
> clients I've tested on IOS, Android, Windows and Mac OS X always
> prepend the extra slash (eg. DELETE /path/to/directory/ which renders
> a 204 that's expected)
>
> - also, after deleting a directory, the screen refreshes making it
> look like the delete was successful. Forcing a refresh will reveal
> that the directory still actually exist. Server logs reveal a 301
> redirect as mentioned.
>
> I was advised by "bcooksley" to email this list to get someguidance as
> to what the correct behavior should be and whether to file a bug
> report.
>
> I have a few test servers listed on
> https://www.pixi.me/webdav.php#targets ready for anyone who has some
> spare time to help reproduce my issues.
Does a patch like this fix the issue?
--
David Faure, [email protected], http://www.davidfaure.fr
Working on KDE, in particular KDE Frameworks 5
diff --git a/kioslave/http/http.cpp b/kioslave/http/http.cpp
index 42ccc94..b13eed1 100644
--- a/kioslave/http/http.cpp
+++ b/kioslave/http/http.cpp
@@ -1443,8 +1443,12 @@ void HTTPProtocol::rename( const KUrl& src, const KUrl& dest, KIO::JobFlags flag
davError();
}
-void HTTPProtocol::del( const KUrl& url, bool )
+void HTTPProtocol::del(const KUrl& _url, bool isFile)
{
+ KUrl url(_url);
+ if (!isFile) {
+ url.adjustPath(KUrl::AddTrailingSlash);
+ }
kDebug(7113) << url;
if (!maybeSetRequestUrl(url))
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
