Skip to content

Refactor validation script to use dompurify#38

Merged
wiz merged 1 commit intomasterfrom
knorrium/refactor_validator
Jan 4, 2026
Merged

Refactor validation script to use dompurify#38
wiz merged 1 commit intomasterfrom
knorrium/refactor_validator

Conversation

@knorrium
Copy link
Member

@knorrium knorrium commented Jan 4, 2026

Used a few malicious SVGs to test

> mining-pool-logos@1.0.0 validate
> node check_logos.js

Unsafe content found in: 1.svg
  - Found javascript: URLs
Unsafe content found in: 2.svg
  - Found javascript: URLs
Unsafe content found in: svgavatar.svg
  - Found 1 script tag(s)
  - DOMPurify removed 1 script element(s)

When everything passes

> mining-pool-logos@1.0.0 validate
> node check_logos.js

No unsafe content found in SVG files

@knorrium knorrium requested a review from wiz January 4, 2026 16:36
@wiz wiz merged commit 89844ba into master Jan 4, 2026
3 checks passed
@knorrium knorrium deleted the knorrium/refactor_validator branch January 4, 2026 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wiz wiz wiz approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@knorrium @wiz