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

Reply via email to