Title: [267693] releases/WebKitGTK/webkit-2.30/Source/WebCore
- Revision
- 267693
- Author
- [email protected]
- Date
- 2020-09-28 01:18:13 -0700 (Mon, 28 Sep 2020)
Log Message
Merge r267620 - [SOUP] Slack.com is not working, new messages do not load due to WebSocket authentication failure issue
https://bugs.webkit.org/show_bug.cgi?id=149551
Reviewed by Michael Catanzaro.
The problem is that we are not including all the cookies in the web socket request. There are two cookies with
the Lax same-site policy set, that are not included for the websocket request.
* Modules/websockets/ThreadableWebSocketChannel.cpp:
(WebCore::ThreadableWebSocketChannel::webSocketConnectRequest): Add same site information to WebSocket requests.
Modified Paths
Diff
Modified: releases/WebKitGTK/webkit-2.30/Source/WebCore/ChangeLog (267692 => 267693)
--- releases/WebKitGTK/webkit-2.30/Source/WebCore/ChangeLog 2020-09-28 06:36:34 UTC (rev 267692)
+++ releases/WebKitGTK/webkit-2.30/Source/WebCore/ChangeLog 2020-09-28 08:18:13 UTC (rev 267693)
@@ -1,3 +1,16 @@
+2020-09-26 Carlos Garcia Campos <[email protected]>
+
+ [SOUP] Slack.com is not working, new messages do not load due to WebSocket authentication failure issue
+ https://bugs.webkit.org/show_bug.cgi?id=149551
+
+ Reviewed by Michael Catanzaro.
+
+ The problem is that we are not including all the cookies in the web socket request. There are two cookies with
+ the Lax same-site policy set, that are not included for the websocket request.
+
+ * Modules/websockets/ThreadableWebSocketChannel.cpp:
+ (WebCore::ThreadableWebSocketChannel::webSocketConnectRequest): Add same site information to WebSocket requests.
+
2020-09-10 Carlos Garcia Campos <[email protected]>
[GTK][WPE] Support NTLM authentication
Modified: releases/WebKitGTK/webkit-2.30/Source/WebCore/Modules/websockets/ThreadableWebSocketChannel.cpp (267692 => 267693)
--- releases/WebKitGTK/webkit-2.30/Source/WebCore/Modules/websockets/ThreadableWebSocketChannel.cpp 2020-09-28 06:36:34 UTC (rev 267692)
+++ releases/WebKitGTK/webkit-2.30/Source/WebCore/Modules/websockets/ThreadableWebSocketChannel.cpp 2020-09-28 08:18:13 UTC (rev 267693)
@@ -33,6 +33,7 @@
#include "ContentRuleListResults.h"
#include "Document.h"
+#include "FrameLoader.h"
#include "HTTPHeaderValues.h"
#include "Page.h"
#include "RuntimeEnabledFeatures.h"
@@ -115,6 +116,7 @@
request.setAllowCookies(validatedURL->areCookiesAllowed);
request.setFirstPartyForCookies(document.firstPartyForCookies());
request.setHTTPHeaderField(HTTPHeaderName::Origin, document.securityOrigin().toString());
+ FrameLoader::addSameSiteInfoToRequestIfNeeded(request, &document);
// Add no-cache headers to avoid compatibility issue.
// There are some proxies that rewrite "Connection: upgrade"
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
