Skip to content

Further clarify post-request check #730

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 6, 2025
Merged

Further clarify post-request check #730

merged 4 commits into from
Jun 6, 2025

Conversation

TimvdLippe
Copy link
Contributor

@TimvdLippe TimvdLippe commented May 30, 2025

It was inconsistent with the pre-request check with regards to the handling of strict-dynamic in both case-sensitivity and parser metadata.

Additionally, add a clarifying comment about why the post-request check also checks the request, to avoid confusion for implementers.

Fixes #727
Part of #728


Preview | Diff

It was inconsistent with the pre-request check with regards to the
handling of `strict-dynamic` in both case-sensitivity and parser
metadata.

Additionally, add a clarifying comment about why the post-request
check also checks the request, to avoid confusion for implementers.

Fixes w3c#727 
Part of w3c#728
Copy link
Member

@antosart antosart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks and sorry for the delay here!

@antosart antosart added the editorial Changes that do not affect how the standard is understood label Jun 5, 2025
@w3cbot
Copy link

w3cbot commented Jun 6, 2025

antosart marked as non substantive for IPR from ash-nazg.

@antosart antosart merged commit 7690298 into w3c:main Jun 6, 2025
2 checks passed
github-actions bot added a commit that referenced this pull request Jun 6, 2025
SHA: 7690298
Reason: push, by antosart

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@TimvdLippe TimvdLippe deleted the patch-1 branch June 6, 2025 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
editorial Changes that do not affect how the standard is understood
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Inconsistency between script pre- and post-request checks for strict-dynamic ASCII case-sensitivity
3 participants