[Webkit-unassigned] [Bug 126384] [SOUP] WebSockets must use system proxy settings
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Jan 8 09:19:41 PST 2018
https://bugs.webkit.org/show_bug.cgi?id=126384
Michael Catanzaro <mcatanzaro at igalia.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #330682|review? |review+
Flags| |
--- Comment #31 from Michael Catanzaro <mcatanzaro at igalia.com> ---
Comment on attachment 330682
--> https://bugs.webkit.org/attachment.cgi?id=330682
Patch with unit tests
View in context: https://bugs.webkit.org/attachment.cgi?id=330682&action=review
r=me because I can't find any problems with the code, and I like the new test. But I'm still concerned about not blocking WebSocket connections when a proxy is in use in the fallback codepath.
> Tools/TestWebKitAPI/Tests/WebKitGLib/TestWebKitWebContext.cpp:700
> + void webSockedConnected(WebSocketServerType serverType)
webSocketConnected
> Tools/TestWebKitAPI/glib/WebKitGLib/WebKitTestServer.cpp:56
> +#if SOUP_CHECK_VERSION(2, 50, 0)
Why do we switch to (2, 50, 0) here? What requires libsoup 2.50.0?
> Tools/TestWebKitAPI/glib/WebKitGLib/WebKitTestServer.cpp:117
> + soup_uri_free(uri);
Why are you not able to use GUniquePtr<SoupURI>?
> Tools/TestWebKitAPI/glib/WebKitGLib/WebKitTestServer.h:55
> + SoupURI* m_baseURI { nullptr };
> +#if SOUP_CHECK_VERSION(2, 50, 0)
> + SoupURI* m_baseWebSocketURI { nullptr };
> +#endif
I'd convert these to use GUniquePtr.
--
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20180108/d62369ed/attachment.html>
More information about the webkit-unassigned
mailing list