Make WordPress Core

Opened 11 months ago

#48054 new enhancement

wp.sanitize.stripTags should iterate instead of using recursion

Reported by: jrchamp Owned by:
Milestone: Awaiting Review Priority: normal
Severity: minor Version: 5.2.3
Component: General Keywords: has-patch
Focuses: Cc:


The changes to wp-sanitize.js seemed wasteful and like it produced dead code in the final return.


When I got into it to fix the dead code, I also felt a way about the recursion, because stack and memory exhaustion are two other potential issues. So this way should *at most* be one function call and only duplicate memory instead of being O(n*memory).


This is probably a drive-by commit and for that I apologize. I hope it's more useful than not, but your mileage may vary. Thank you for making WordPress.

Change History (0)

Note: See TracTickets for help on using tickets.