Skip to content

security: disable implicit remote URL fetching (SSRF v0.2.0) - #1

Merged
tmolavi merged 1 commit into
mainfrom
security/ssrf-v0.2.0-disable-url-fetch
Oct 25, 2026
Merged

tmolavi merged 1 commit into
mainfrom
security/ssrf-v0.2.0-disable-url-fetch

Conversation

@tmolavi

@tmolavi tmolavi commented Oct 15, 2026

Copy link
Copy Markdown
Owner

Summary

Temporarily disables implicit remote URL fetching in extract_website_context to prevent SSRF through direct URL input and URL-shaped topics.

Changes

  • Keep URL-derived domain/brand heuristics for prompt generation.
  • Make remote metadata fetching explicit opt-in via fetch_remote=True.
  • Preserve topic-based generation and local export/file ingestion.
  • Add regression tests proving no network call occurs by default and opt-in behavior remains explicit.

Verification

  • python3 -m pytest tests/ -q
  • Result: 10 passed

Scope

  • Review-only; do not merge automatically.
  • No Hermes, server, or paid-service changes.

@tmolavi
tmolavi merged commit a1e6606 into main Oct 25, 2026
3 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