Skip to content

Fix/security holes - #8

Merged
technoporada merged 4 commits into
mainfrom
fix/security-holes
Nov 2, 2026
Merged

technoporada merged 4 commits into
mainfrom
fix/security-holes

Conversation

@technoporada

Copy link
Copy Markdown
Owner

No description provided.

- link_checker.py: SSRF guard - wewnętrzne URL nie są wysyłane do VirusTotal (dodano _is_internal_url)
- app.py: wymagany admin token na /api/stats i /api/ratelimit
- app.py: dodano CORSMiddleware (tylko localhost)
- database.py: limit 10000 scanów i 1000 raportów - czyszczenie najstarszych przy przekroczeniu (DoS guard)
- frontend/index.html: generateAbuseReport używa textContent + escapeHTML zamiast innerHTML z data.body (XSS przez source_ip/domain/url/evidence)
- app.py: wymagany admin token na /api/email/parse, /api/link/check, /api/phone/check, /api/abuse/generate
- app.py: dodano CORSMiddleware (tylko localhost)
- database.py: limit 10000 scanów i 1000 raportów (DoS guard)
- link_checker.py: SSRF guard dla VirusTotal (wewnętrzne URL nie są wysyłane)
- link_checker.py: follow_redirects=False wszędzie
- link_checker.py: sanitizacja danych z zewnętrznych API (XSS guard)
- frontend/index.html: XSS w generateAbuseReport (textContent + escapeHTML)
- app.py: wymagany admin token na /api/abuse/templates (było bez auth)
- database.py: _enforce_db_size — czyszczenie najstarszych scanów przy >5000 rekordów (DoS guard)
@technoporada
technoporada merged commit 96d0184 into main Nov 2, 2026
9 checks passed
@technoporada
technoporada deleted the fix/security-holes branch November 2, 2026 05:53
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.

2 participants