Skip to content

Move Integrity-Policy signature-based SRI tests to tentative #52924

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jun 4, 2025

Conversation

yoavweiss
Copy link
Contributor

@yoavweiss yoavweiss commented Jun 3, 2025

Since Signature-based SRI does not yet have consensus around it, tests of the feature should be tentative.

This PR splits such a test case from the Integrity-Policy tests to its own tentative test.

@yoavweiss yoavweiss requested a review from antosart June 3, 2025 10:05
@yoavweiss yoavweiss requested a review from antosart June 3, 2025 15:33
@yoavweiss yoavweiss merged commit 24e2df1 into master Jun 4, 2025
17 checks passed
@yoavweiss yoavweiss deleted the signature_tentative branch June 4, 2025 07:36
yoavweiss added a commit to yoavweiss/WebKit that referenced this pull request Jun 4, 2025
https://bugs.webkit.org/show_bug.cgi?id=294018

Reviewed by NOBODY (OOPS!).

web-platform-tests/wpt#52924 removes the Signature-based SRI tests from
the rest of the Integrity-Policy tests. This PR imports that.

Upstream commit: web-platform-tests/wpt@6ba4f8f

* LayoutTests/imported/w3c/web-platform-tests/subresource-integrity/integrity-policy/script.https.html: Remove signature test.
webkit-commit-queue pushed a commit to yoavweiss/WebKit that referenced this pull request Jun 4, 2025
https://bugs.webkit.org/show_bug.cgi?id=294018

Reviewed by Sam Weinig.

web-platform-tests/wpt#52924 removes the Signature-based SRI tests from
the rest of the Integrity-Policy tests. This PR imports that.

Upstream commit: web-platform-tests/wpt@6ba4f8f

* LayoutTests/imported/w3c/web-platform-tests/subresource-integrity/integrity-policy/script.https.html: Remove signature test.

Canonical link: https://commits.webkit.org/295823@main
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants