Strip html tags from title #1801
Conversation
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #1801 +/- ##
=======================================
Coverage 70.42% 70.42%
=======================================
Files 23 23
Lines 2597 2597
Branches 594 595 +1
=======================================
Hits 1829 1829
Misses 661 661
Partials 107 107
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
src/util/misc.ts
Outdated
| @@ -283,7 +283,7 @@ export function getStrippedTitleFromHtml(html: string) { | |||
| const doc = domino.createDocument(html) | |||
There was a problem hiding this comment.
I'm choqued by this line which is super CPU expensive. There is no way we coukd do that quicker?! I know this is not the topic of this PR, but stil...
tests added
| } | ||
| return title.replace(/<[^>]*>?/gm, '') |
Check failure
Code scanning / CodeQL
Incomplete multi-character sanitization
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
Strip HTML tags from the title to avoid HTML tags in the search results
fix: #1797