Skip to content

[HtmlSanitizer] Fix method names #21216

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

Conversation

OskarStark
Copy link
Contributor

Changed incorrect method names from allowed* to allow*:

  • allowedLinkSchemes() → allowLinkSchemes()
  • allowedLinkHosts() → allowLinkHosts()
  • allowedMediaSchemes() → allowMediaSchemes()
  • allowedMediaHosts() → allowMediaHosts()

These are the correct method names used in the HtmlSanitizerConfig class.

Fixes #21211

@OskarStark OskarStark requested a review from javiereguiluz July 18, 2025 07:36
@OskarStark OskarStark added the bug label Jul 18, 2025
@carsonbot carsonbot added this to the 6.4 milestone Jul 18, 2025
@carsonbot carsonbot changed the title Fix HTML Sanitizer method names in documentation [HtmlSanitizer] Fix HTML Sanitizer method names in documentation Jul 18, 2025
@OskarStark OskarStark changed the title [HtmlSanitizer] Fix HTML Sanitizer method names in documentation [HtmlSanitizer] Fix method names Jul 18, 2025
@OskarStark OskarStark requested review from xabbuh and stof July 23, 2025 09:13
@OskarStark OskarStark force-pushed the html-sanitizer-method-names-6.4 branch from f2cd0d7 to 0918b69 Compare July 23, 2025 09:23
@OskarStark OskarStark merged commit 4e9bdb1 into 6.4 Jul 23, 2025
6 checks passed
@OskarStark OskarStark deleted the html-sanitizer-method-names-6.4 branch July 23, 2025 09:23
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.

4 participants