Sanitize blog post HTML#34
Conversation
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
|
Quick note on the Socket advisory comments: they come from sanitize-html's standard parser dependencies (htmlparser2/entities). I kept a maintained sanitizer here rather than a custom parser/regex because this PR is specifically closing an XSS sink. The Socket checks are successful, and I can switch sanitizer approach if you prefer another package. |
Summary
dangerouslySetInnerHTMLsanitize-htmlas an explicit web dependencyFixes #15
Checks
pnpm --filter @c0upons/web run typecheckpnpm --filter @c0upons/web exec eslint 'app/blog/[slug]/page.tsx' lib/sanitize-blog-html.tspnpm --filter @c0upons/web run buildNote: full app lint still reports pre-existing errors outside this change in
about,home,privacy,terms,offline, andapi/webhooks/crawlproof.