Skip to content

Fix SanitizerConfig API typo #40471

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 1 commit into from
Jul 21, 2025
Merged

Fix SanitizerConfig API typo #40471

merged 1 commit into from
Jul 21, 2025

Conversation

captainbrosset
Copy link
Contributor

Description

The SanitizerConfig can be used by both setHTML and setHTMLUnsafe (as well as parseHTML and parseHTMLUnsafe), however the intro repeated twice the same method names.

Motivation

Fixing a typo.

Additional details

The same fix was already done in the Sanitizer API in #40461.

Related issues and pull requests

#40461.

The SanitizerConfig can be used by both setHTML and setHTMLUnsafe (as well as parseHTML and parseHTMLUnsafe), however the intro repeated twice the same method names.
@captainbrosset captainbrosset requested a review from a team as a code owner July 21, 2025 14:39
@captainbrosset captainbrosset requested review from sideshowbarker and removed request for a team July 21, 2025 14:39
@github-actions github-actions bot added Content:WebAPI Web API docs size/s [PR only] 6-50 LoC changed labels Jul 21, 2025
Copy link
Contributor

Preview URLs

Copy link
Collaborator

@hamishwillee hamishwillee left a comment

Choose a reason for hiding this comment

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

Thanks @captainbrosset - I can't believe I injected that error twice. Sorry.

@hamishwillee hamishwillee merged commit 7141d3f into main Jul 21, 2025
11 checks passed
@hamishwillee hamishwillee deleted the captainbrosset-patch-1 branch July 21, 2025 23:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:WebAPI Web API docs size/s [PR only] 6-50 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants