Skip to content
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

[Spec] Align with IETF spec on some details #1371

Merged
merged 1 commit into from
Dec 19, 2024
Merged

Conversation

qingxinwu
Copy link
Collaborator

@qingxinwu qingxinwu commented Dec 18, 2024

for B&A debug reports handling:

  1. Add check about if debug report url is https in w3c spec
  2. [=component win debug reports=][key] is a single url, not a list of urls. Although the w3c spec defined it as a list, but was used as a single URL anyways. Change to align with IETF spec as well.

Preview | Diff

@qingxinwu qingxinwu added the spec Relates to the spec label Dec 18, 2024
@qingxinwu qingxinwu changed the title align with IETF spec on some details [Spec] Align with IETF spec on some details Dec 18, 2024
@qingxinwu
Copy link
Collaborator Author

@brusshamilton PTAL. Thanks!

@JensenPaul JensenPaul merged commit 62fdc90 into WICG:main Dec 19, 2024
2 checks passed
github-actions bot added a commit that referenced this pull request Dec 19, 2024
SHA: 62fdc90
Reason: push, by JensenPaul

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@qingxinwu qingxinwu deleted the fix branch December 20, 2024 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spec Relates to the spec
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants