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

Update the default handling #243

Merged
merged 6 commits into from
Dec 12, 2024
Merged

Update the default handling #243

merged 6 commits into from
Dec 12, 2024

Conversation

otherdaniel
Copy link
Collaborator

@otherdaniel otherdaniel commented Nov 29, 2024

SetHTMLOptions's "sanitizer" key can now also contain an enum SanitizerPresets, which has only one member, "default". Which, it turns out, is the default value. This is now the only way to invoke the defaults. An empty dictionary would then
be a Sanitizer which allows anything. (Minus the "removeUnsafe" content, when used with a "safe" method.)

This follows discussion at the 2024-11-27 meeting.

The explainer still needs updaing.


Preview | Diff

index.bs Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
index.bs Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
otherdaniel and others added 2 commits December 10, 2024 15:24
Co-authored-by: Anne van Kesteren <[email protected]>
Co-authored-by: Anne van Kesteren <[email protected]>
Copy link
Collaborator

@annevk annevk left a comment

Choose a reason for hiding this comment

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

Left some additional nits, but generally this looks great to me. Thanks!

index.bs Outdated Show resolved Hide resolved
@otherdaniel otherdaniel merged commit 9254522 into WICG:main Dec 12, 2024
2 checks passed
github-actions bot added a commit that referenced this pull request Dec 12, 2024
SHA: 9254522
Reason: push, by otherdaniel

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants