Bug 191652 – [Curl] Load HTTP body of 401 response when AuthenticationChange is cancelled.

• Status: RESOLVED
• Resolution: FIXED
• Component: Platform

[Basuke Suzuki]

Currently empty body.

[Takashi Komori]

Created attachment 361772 [details]
Patch

[Takashi Komori]

When server returns 401 error WinCairo port doesn't print any result.
When 401 error we should call didReceiveResponse.
This patch fix it.

[EWS Watchlist]

Comment on attachment 361772 [details]
Patch

Attachment 361772 [details] did not pass mac-debug-ews (mac):
Output: https://webkit-queues.webkit.org/results/11119078

New failing tests:
http/tests/inspector/network/resource-initiatorNode.html

[EWS Watchlist]

Created attachment 361780 [details]
Archive of layout-test-results from ews112 for mac-highsierra

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews112  Port: mac-highsierra  Platform: Mac OS X 10.13.6

[Alex Christensen]

Does this fix some layout tests?  That should be mentioned in the ChangeLog.

[Takashi Komori]

Created attachment 361873 [details]
Patch

[Takashi Komori]

(In reply to Alex Christensen from comment #5)
> Does this fix some layout tests?  That should be mentioned in the ChangeLog.

This patch doesn't fix existing layouttest.
This fixes the behavior of server returning 401, and testrunner can't handle it.
Because testrunner challenges authentication infinitely when auth fails.

Is there apropriate test for this situation?

[Alex Christensen]

(In reply to Takashi Komori from comment #7)
> Is there apropriate test for this situation?

There are many tests where servers return 401.  If there isn't a test that this change fixes, there should be.  I'll be there's already at least one, and it should be mentioned.

[Takashi Komori]

Created attachment 362976 [details]
Patch

[Takashi Komori]

(In reply to Alex Christensen from comment #8)
Thank you for your comment.

New patch passes http/tests/security/401-logout/401-logout.php

Expected results for WebKitTestRunner and DumpRenderTree are different.
This is because difference of didReceiveAuthenticationChallenge implementations between TestRunner and DRT.

On WIncairo port, we should use LayoutTests/http/tests/security/401-logout/401-logout-expected.txt as DumpRenderTree's expected result,
and use LayoutTests/platform/wk2/http/tests/security/401-logout/401-logout-expected.txt as WebKitTestRunner's expected result.

[WebKit Commit Bot]

Comment on attachment 362976 [details]
Patch

Clearing flags on attachment: 362976

Committed r242092: <https://trac.webkit.org/changeset/242092>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/48405043>