[Webkit-unassigned] [Bug 222558] HTTP method in web inspector network tab is not what WebKit actually sent after a redirect from POST to GET

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=222558

Devin Rousso <drousso at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |drousso at apple.com

--- Comment #18 from Devin Rousso <drousso at apple.com> ---
Comment on attachment 426090
  --> https://bugs.webkit.org/attachment.cgi?id=426090
WIP

View in context: https://bugs.webkit.org/attachment.cgi?id=426090&action=review

Nice catch!

(In reply to Michael Catanzaro from comment #17)
> Since the problem reportedly also affects Safari, maybe a web inspector developer would have time to help with a proper test?

The only Web Inspector test that I know of for redirects (or more like that involves redirects) is `LayoutTests/http/tests/inspector/network/resource-timing.html`, specifically the `"Resource.TimingData.Redirect"` test case.  You could probably do something like that that looks at the resulting `resource.requestMethod` (and/or other properties) after a redirect that changes it.

> Source/WebInspectorUI/UserInterface/Models/Resource.js:681
> -        this._redirects.push(new WI.Redirect(oldURL, request.method, oldHeaders, response.status, response.statusText, response.headers, elapsedTime));
> +        this._requestMethod = request.method || null;
> +        this._redirects.push(new WI.Redirect(oldURL, oldMethod, oldHeaders, response.status, response.statusText, response.headers, elapsedTime));

I wonder if maybe we should also have `oldStatus` and `oldStatusText` too, just to be safe ��

-- 
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/20210426/d4ea2e06/attachment.htm>