-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Update HTML Sanitizer doc for max_input_length
option
#18147
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 HTML Sanitizer doc for max_input_length
option
#18147
Conversation
Hey! Oh no, it looks like you have made this PR towards a branch that is not maintained anymore. :/ Cheers! Carsonbot |
4434689
to
b4acb94
Compare
friendly ping @tgalopin |
Do you know why the "CI/Code Blocks" fails ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You could also explain why if you want (it's because it's a source of DoS attacks)
Co-authored-by: Oskar Stark <oskarstark@googlemail.com>
…(t.le-gacque, tristan-lg) This PR was merged into the 6.2 branch. Discussion ---------- Update HTML Sanitizer doc for `max_input_length` option Add missing documentation for https://symfony.com/doc/current/html_sanitizer.html#allow-elements Related to the bug symfony/symfony#49040 Commits ------- 370b3d9 Apply suggestions from code review 7011d17 Update HTML Sanitizer doc for `max_input_length` option 6d2e21c Update HTML Sanitizer doc for `max_input_length` option ce02cd4 Update HTML Sanitizer doc for `max_input_length` option
Tristan, thanks and congrats on a great first Symfony Docs contribution 🎉 Thanks to reviewers too 🙌 |
Closing manually as merged. GitHub shows it as "closed", but it was merged; see e39cbf6 |
Add missing documentation for https://symfony.com/doc/current/html_sanitizer.html#allow-elements
Related to the bug symfony/symfony#49040