QNetworkCookie - ignore unknown attributes

RFC6265 clarifies that unknown cookie attributes should be ignored,
including the version attribute which was defined by RFC2109 but
not used correctly in practice.

This fixes case 0008 in the test suite with minimal risk.

Task-number: QTBUG-15794
Change-Id: I6f15e8e5e2e5f1ed168fc733a5c84d606a452252
Reviewed-by: Richard J. Moore <rich@kde.org>
This commit is contained in:
Shane Kearns 2012-06-07 15:28:26 +01:00 committed by Qt by Nokia
parent a32d6834ad
commit 57adc1761d

View File

@ -1025,16 +1025,8 @@ QList<QNetworkCookie> QNetworkCookiePrivate::parseSetCookieHeaderLine(const QByt
cookie.setSecure(true); cookie.setSecure(true);
} else if (field.first == "httponly") { } else if (field.first == "httponly") {
cookie.setHttpOnly(true); cookie.setHttpOnly(true);
} else if (field.first == "comment") {
//cookie.setComment(QString::fromUtf8(field.second));
} else if (field.first == "version") {
if (field.second != "1") {
// oops, we don't know how to handle this cookie
return result;
}
} else { } else {
// got an unknown field in the cookie // ignore unknown fields in the cookie (RFC6265 section 5.2, rule 6)
// what do we do?
} }
position = nextNonWhitespace(cookieString, position); position = nextNonWhitespace(cookieString, position);