AuroraMiddleware/qt5base-lts
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Revert "HTTP/2 protocol handler: set the redirect URL on reply"

Browse Source 
This reverts commit 9e268185bb.
Reasons: must go into 5.9 + the logic in this patch needs fixing, not good as
it is now.

Change-Id: I92516ddb1d2661d85ce77c54a1307f45df1ab635
Reviewed-by: Andy Shaw <andy.shaw@qt.io>
Reviewed-by: Edward Welbourne <edward.welbourne@qt.io>
This commit is contained in:
Timur Pocheptsov 2017-09-29 04:17:27 +00:00
parent 56a3cbc5c7
commit 617ba6f1c4
1 changed files with 1 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
src/network/access/qhttp2protocolhandler.cpp
View File

@ -1042,18 +1042,12 @@ void QHttp2ProtocolHandler::updateStream(Stream &stream, const HPack::HttpHeader
}
const auto httpReplyPrivate = httpReply->d_func();
int statusCode = 0;
for (const auto &pair : headers) {
const auto &name = pair.name;
auto value = pair.value;
if (name == ":status") {
// TODO: part of this code copies what SPDY does when processing
// headers. It would be nice to re-factor HTTP/2 protocol handler
// to make it more strict in extracting things. For example, the
// status code below ... yeah, should be an integer, right?
statusCode = value.left(3).toInt();
httpReply->setStatusCode(statusCode);
httpReply->setStatusCode(value.left(3).toInt());
httpReplyPrivate->reasonPhrase = QString::fromLatin1(value.mid(4));
} else if (name == ":version") {
httpReplyPrivate->majorVersion = value.at(5) - '0';
@ -1064,17 +1058,6 @@ void QHttp2ProtocolHandler::updateStream(Stream &stream, const HPack::HttpHeader
if (ok)
httpReply->setContentLength(length);
} else {
if (statusCode > 300 && statusCode <= 308 && name == "location") {
// For HTTP/1 'location' is handled (and redirect URL set) when
// a protocol handler emits channel->allDone(). Http/2 protocol
// handler never emits allDone, since we have many requests
// multiplexed in one channel at any moment and we are never
// done :) So it's time to extract location.
const QUrl redirectUrl(QUrl::fromEncoded(value));
if (redirectUrl.isValid())
httpReply->setRedirectUrl(redirectUrl);
}
QByteArray binder(", ");
if (name == "set-cookie")
binder = "\n";