Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Tags whitelist support #536

Closed
wants to merge 1 commit into from
Closed

Tags whitelist support #536

wants to merge 1 commit into from

Conversation

mloginov
Copy link

options: {
sanitize: true,
allowedTags: 'iframe|img|video'
}

For #133

@chjj
Copy link
Member

chjj commented May 8, 2015

Please refactor this so a RegExp is not dynamically created each call. A hash table check would be faster than a regex: reduce an array of tags to an object and check the object's property.

@paibamboo
Copy link

please continue :)

@joshbruce
Copy link
Member

joshbruce commented Dec 3, 2017

I know this is old, see #956

Closing:

  1. Never refactored.
  2. Conflicts.
  3. Missing tests.
  4. Stale.
  5. Added whitelist tag support to work alongside sanitizer #710

@joshbruce joshbruce closed this Dec 3, 2017
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants