Skip to content

dealsubs: try the sites with a probe-verified form first - #133

Merged
ralyodio merged 1 commit into
masterfrom
chore/dealsubs-order
Oct 6, 2026
Merged

ralyodio merged 1 commit into
masterfrom
chore/dealsubs-order

Conversation

@ralyodio

@ralyodio ralyodio commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

The first live run spent its three tries on homepages with no form. The ten probe-verified sites now lead the catalog. Only the order changes.

🤖 Generated with Claude Code

The first live run spent its three tries on homepages with no form. The ten
sites where `dealsubs probe --all` found an email box and a way to submit
now lead the catalog.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Oct 6, 2026

Copy link
Copy Markdown

ThreatCrush Security Scan

32 finding(s)

HIGH/CRITICAL: 7 | MEDIUM: 12 | LOW: 13

Severity Rule Location
HIGH py-ssrf-outbound-request dev2/dev2-site:147
HIGH py-ssrf-outbound-request dev2/dev2-site:876
HIGH py-ssrf-outbound-request dev2/dev2-site:1565
HIGH sh-remote-script-execution root-ubuntu.sh:3236
HIGH sh-remote-script-execution root-ubuntu.sh:3237
HIGH sh-remote-script-execution root-ubuntu.sh:5071
HIGH sh-remote-script-execution root-ubuntu.sh:5075
MEDIUM sql-template-interpolation dev2/dev2-site:1102
MEDIUM sql-template-interpolation dev2/dev2-site:1179
MEDIUM sh-remote-script-execution root-ubuntu.sh:5248
MEDIUM redos-nested-quantifier src/domain-free.ts:56
MEDIUM redos-nested-quantifier src/emoji.ts:167
MEDIUM redos-nested-quantifier src/icon.ts:166
MEDIUM redos-nested-quantifier src/mail.ts:1042
MEDIUM sql-template-interpolation src/users-dump.ts:487
MEDIUM sql-string-concatenation src/users-dump.ts:507
MEDIUM sql-template-interpolation src/users-dump.ts:540
MEDIUM sql-string-concatenation src/users-dump.ts:574
MEDIUM redos-nested-quantifier src/wcag.ts:556
LOW secret-generic-credential src/credentials.ts:36
LOW secret-generic-credential src/credentials.ts:53
LOW secret-generic-credential src/credentials.ts:56
LOW secret-generic-credential src/user-export.ts:632
LOW secret-generic-credential src/user-export.ts:638
LOW secret-generic-api-key test/credentials.test.ts:208
LOW secret-generic-credential test/mail.test.ts:141
LOW secret-generic-credential test/proxy.test.ts:52
LOW secret-generic-credential test/proxy.test.ts:68
LOW secret-generic-credential test/shorten.test.ts:36
LOW secret-database-url test/users-dump.test.ts:108
LOW secret-database-url test/users-dump.test.ts:119
LOW secret-database-url test/users-dump.test.ts:120

Snippets are redacted; ThreatCrush never prints matched credential material.

@ralyodio
ralyodio merged commit 3652663 into master Oct 6, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant