[HtmlSanitizer] Add docs for the new HtmlSanitizer defaultAction config#20019

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Open

Seldaek wants to merge1 commit intosymfony:7.2

base:7.2

Choose a base branch

fromSeldaek:patch-10

Open

[HtmlSanitizer] Add docs for the new HtmlSanitizer defaultAction config#20019

Seldaek wants to merge1 commit intosymfony:7.2fromSeldaek:patch-10

+74 −0

Conversation

Copy link

Member

Seldaek commentedJul 3, 2024•
edited
Loading

Fixes#20004

Requires#20019 to be merged for completeness.

Add docs for the new HtmlSanitizer defaultAction config

c146eb8

carsonbot added the Status: Needs Review label

Jul 3, 2024

carsonbot added this to the7.2 milestone

Jul 3, 2024

Copy link

Neirda24 commentedJul 3, 2024

Hi@Seldaek . Yes I am. did not had the time to finish things yet.

javiereguiluz added the HtmlSanitizer label

Jul 3, 2024

carsonbot changed the title~~Add docs for the new HtmlSanitizer defaultAction config~~[HtmlSanitizer] Add docs for the new HtmlSanitizer defaultAction config

Jul 3, 2024

javiereguiluz added the On hold label

Jul 3, 2024

Neirda24 mentioned this pull request

Jul 4, 2024

[FrameworkBundle] Allow default action in configurationsymfony/symfony#57653

Open

Copy link

@Seldaek : here you go :symfony/symfony#57653 was pretty straightforward. I'll take time to add some tests on DI as well because there are none at the moment. Should it be in the same PR or another one to keep things separated ?