Bug 260107

Summary:

ASSERTION FAILED: isAllowedByContentSecurityPolicy(request.url(), ContentSecurityPolicy::RedirectResponseReceived::No) seen with multiple tests under http/wpt/workers/modules/

Product:

WebKit

Reporter:

Ben Schwartz <ben_schwartz>

Component:

New Bugs

Assignee:

youenn fablet <youennf>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

Hironori.Fujii, webkit-bot-watchers-bugzilla, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

See Also:

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

Attachments:

Description	Flags
wk1-ventura-arm64-debug-crash-log.txt	none
wk1-ventura-x86_64-debug-crash-log.txt	none
wk2-ventura-arm64-debug-crash-log.txt	none

Description Ben Schwartz 2023-08-11 16:42:28 PDT

http/wpt/workers/modules/dedicated-worker-import-csp.html

This test is a constant failure on all platforms (debug builds).

HISTORY:

https://results.webkit.org/?suite=layout-tests&test=http%2Fwpt%2Fworkers%2Fmodules%2Fdedicated-worker-import-csp.html

CRASH LOGS:

Attached.

CRASH LOG URLS:

https://build.webkit.org/results/Apple-Ventura-Debug-AppleSilicon-WK2-Tests/266817@main%20(3321)/http/wpt/workers/modules/dedicated-worker-import-csp-crash-log.txt
https://build.webkit.org/results/Apple-Ventura-Debug-WK1-Tests/266814@main%20(3200)/http/wpt/workers/modules/dedicated-worker-import-csp-crash-log.txt
https://build.webkit.org/results/Apple-Ventura-Debug-AppleSilicon-WK1-Tests/266824@main%20(3146)/http/wpt/workers/modules/dedicated-worker-import-csp-crash-log.txt

REPRODUCIBILITY:

I was able to reproduce this issue on macOS Ventura Debug 266793@main (latest successful debug build) using the following command:

run-webkit-tests --verbose --debug http/wpt/workers/modules/dedicated-worker-import-csp.html

REGRESSION:

There is no clear regression as this is a WPT test imported at 266801@main.

Comment 1 Radar WebKit Bug Importer 2023-08-11 16:42:38 PDT

<rdar://problem/113777310>

Comment 2 Ben Schwartz 2023-08-11 16:43:00 PDT

Created attachment 467252 [details]
wk1-ventura-arm64-debug-crash-log.txt

Comment 3 Ben Schwartz 2023-08-11 16:43:20 PDT

Created attachment 467253 [details]
wk1-ventura-x86_64-debug-crash-log.txt

Comment 4 Ben Schwartz 2023-08-11 16:43:40 PDT

Created attachment 467254 [details]
wk2-ventura-arm64-debug-crash-log.txt

Comment 5 Ryan Haddad 2023-08-11 17:06:37 PDT

ASSERTION FAILED: isAllowedByContentSecurityPolicy(request.url(), ContentSecurityPolicy::RedirectResponseReceived::No)
/Volumes/Data/worker/Apple-Ventura-Debug-Build/build/Source/WebCore/loader/DocumentThreadableLoader.cpp(148) : WebCore::DocumentThreadableLoader::DocumentThreadableLoader(WebCore::Document &, WebCore::ThreadableLoaderClient &, WebCore::DocumentThreadableLoader::BlockingBehavior, WebCore::ResourceRequest &&, const WebCore::ThreadableLoaderOptions &, RefPtr<WebCore::SecurityOrigin> &&, std::unique_ptr<ContentSecurityPolicy> &&, std::optional<CrossOriginEmbedderPolicy> &&, WTF::String &&, WebCore::DocumentThreadableLoader::ShouldLogError)

Comment 6 EWS 2023-08-11 17:24:58 PDT

Test gardening commit 266839@main (ec6f02c2f324): <https://commits.webkit.org/266839@main>

Reviewed commits have been landed. Closing PR #16631 and removing active labels.

Comment 7 Ben Schwartz 2023-08-11 17:25:14 PDT

Correction in original text: this test is a constant CRASH on all platforms (debug builds), not failure.

Comment 8 EWS 2023-08-11 18:03:43 PDT

Test gardening commit 266841@main (a503aa72c296): <https://commits.webkit.org/266841@main>

Reviewed commits have been landed. Closing PR #16634 and removing active labels.

Comment 9 Ben Schwartz 2023-08-11 18:06:02 PDT

I have marked this test to be skipped while this issue is investigated (PR link: https://github.com/WebKit/WebKit/pull/16634)

Comment 10 Fujii Hironori 2023-08-13 13:09:14 PDT

*** Bug 260128 has been marked as a duplicate of this bug. ***

Comment 11 Ben Schwartz 2023-08-14 14:57:25 PDT

Skipping additional test (http/wpt/workers/modules/dedicated-worker-import-csp.html) under the same bug while the issue is investigated. (PR link: https://github.com/WebKit/WebKit/pull/16681)

Comment 12 EWS 2023-08-14 15:03:16 PDT

Test gardening commit 266878@main (2b25eede2b58): <https://commits.webkit.org/266878@main>

Reviewed commits have been landed. Closing PR #16681 and removing active labels.

Comment 13 youenn fablet 2023-08-22 08:57:29 PDT

Pull request: https://github.com/WebKit/WebKit/pull/16931

Comment 14 EWS 2023-08-23 10:03:36 PDT

Committed 267194@main (0495446c22a5): <https://commits.webkit.org/267194@main>

Reviewed commits have been landed. Closing PR #16931 and removing active labels.