- Issue created by @Anybody
- ๐ฎ๐ณIndia samit.310@gmail.com
samit.310@gmail.com โ made their first commit to this issueโs fork.
- Merge request !243457314: HTML tags stripped for subject โ (Merged) created by samit.310@gmail.com
- Status changed to Needs review
2 months ago 7:58am 8 July 2024 - Status changed to Needs work
about 2 months ago 7:57am 16 July 2024 - ๐ฉ๐ชGermany Anybody Porta Westfalica
@samit.310@gmail.com thanks a lot.
Why
Html::decodeEntities()
?
Why not justDecodes all HTML entities including numerical ones to regular UTF-8 bytes. Double-escaped entities will only be decoded once ("<" becomes "<", not "<"). Be careful when using this function, as it will revert previous sanitization efforts (<script> will become
).strip_tags()
? - ๐ฉ๐ชGermany Anybody Porta Westfalica
Just had a look myself and
Html::decodeEntities()
is correct here, because Drupal escapes email contents itself. So otherwise we'd have double escaping.See ๐ Rename "watchdog_mailer_enable" config key to "enabled" Fixed for details. Thank you!!
-
Anybody โ
committed 0b477437 on 3.x authored by
samit.310@gmail.com โ
Issue #3457314 by Anybody, samit.310@gmail.com: Email title should have...
-
Anybody โ
committed 0b477437 on 3.x authored by
samit.310@gmail.com โ
- Status changed to Fixed
about 2 months ago 1:00pm 16 July 2024 Automatically closed - issue fixed for 2 weeks with no activity.