D24316: Consider the usage of WebDAV methods sufficient for assuming WebDAV
Volker Krause
noreply at phabricator.kde.org
Wed Oct 23 08:25:13 BST 2019
vkrause added a comment.
In D24316#552541 <https://phabricator.kde.org/D24316#552541>, @dfaure wrote:
> Can I just ignore the Depth from the custom http headers instead?
>
> --- i/src/ioslaves/http/http.cpp
> +++ w/src/ioslaves/http/http.cpp
> @@ -172,7 +172,8 @@ static QString sanitizeCustomHTTPHeader(const QString &_header)
> if (!header.contains(QLatin1Char(':')) ||
> header.startsWith(QLatin1String("host"), Qt::CaseInsensitive) ||
> header.startsWith(QLatin1String("proxy-authorization"), Qt::CaseInsensitive) ||
> - header.startsWith(QLatin1String("via"), Qt::CaseInsensitive)) {
> + header.startsWith(QLatin1String("via"), Qt::CaseInsensitive) ||
> + header.startsWith(QLatin1String("depth"), Qt::CaseInsensitive)) {
> continue;
> }
>
Indeed, that should work too, especially considering Depth has its own meta-data field anyway.
REPOSITORY
R241 KIO
REVISION DETAIL
https://phabricator.kde.org/D24316
To: vkrause, dfaure
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191023/fdb570d7/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list