New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WinCairo] include secure cookies for secure WebSockets #9514
[WinCairo] include secure cookies for secure WebSockets #9514
Conversation
Happy to add a test for it if someone lets me know where! I did only find WebSocket tests for cocoa and glib so far. |
EWS run on previous version of this PR (hash 1103bfa) |
cc @fujii |
Source/WebCore/platform/network/curl/NetworkStorageSessionCurl.cpp
Outdated
Show resolved
Hide resolved
{ | ||
// FIXME: This should filter secure cookies out if the caller requests it. | ||
return { cookiesForSession(*this, firstParty, url, false), false }; | ||
return { cookiesForSession(*this, firstParty, url, false, includeSecureCookies), false }; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The second argument is used for document.setSecureCookiesAccessed
. It seems important. https://commits.webkit.org/192473@main It'd be nice, if you properly calculate the second argument in this time.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can follow-up on this one since its not related to the bug report and I didn't find an easy way of doing it.
EWS run on current version of this PR (hash af31d23) |
https://bugs.webkit.org/show_bug.cgi?id=251581 Reviewed by Fujii Hironori. Before this patch for wss:// connections, secure cookies were not includes. This fixes it and includes them. Canonical link: https://commits.webkit.org/259975@main
Committed 259975@main (11421fd): https://commits.webkit.org/259975@main Reviewed commits have been landed. Closing PR #9514 and removing active labels. |
11421fd
af31d23
π§ͺ api-gtk