Summary: | [WPE][GTK] Bogus Response.{status,ok} for successful fetch() requests to a custom URI scheme handler | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Adrian Perez <aperez> | ||||
Component: | WPE WebKit | Assignee: | Adrian Perez <aperez> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | achristensen, berto, bugs-noreply, cgarcia, ews-watchlist, gustavo | ||||
Priority: | P2 | ||||||
Version: | WebKit Local Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=203273 | ||||||
Attachments: |
|
Description
Adrian Perez
2021-02-26 06:58:08 PST
This is somewhat related to bug #203273 but much, much smaller in scope, and I think setting the status to “200 OK” when an URI scheme request is finished with a valid “GInputStreams” should not be controversial, so let's tackle this small annoyance first. Created attachment 421653 [details]
Patch
Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See https://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API Committed r273618: <https://commits.webkit.org/r273618> All reviewed patches have been landed. Closing bug and clearing flags on attachment 421653 [details]. Comment on attachment 421653 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=421653&action=review > Source/WebKit/ChangeLog:8 > + No new tests needed. This should be easy to add a unit test for, especially if you care about not regressing this functionality in the future. We have recently added api to set the response, including api tests for checking the status and message. |