[search] Reconcile diverged content.py (#784) #845

Closed

sleepy wants to merge 0 commits from fix/dedup-search-content-784 into main

sleepy commented

2026-06-03 21:05:26 +02:00

Owner

Problem

services/search/content.py and src/search/content.py were diverged copies (issue #784). The services/ version was missing security improvements.

Fix

Replace services/search/content.py with re-exports from canonical src.search.content.

Testing

66/66 security regression tests pass
528/528 non-pre-existing tests pass

## Problem services/search/content.py and src/search/content.py were diverged copies (issue #784). The services/ version was missing security improvements. ## Fix Replace services/search/content.py with re-exports from canonical src.search.content. ## Testing - 66/66 security regression tests pass - 528/528 non-pre-existing tests pass

sleepy added 1 commit

2026-06-03 21:05:26 +02:00

[search] Reconcile diverged content.py (#784 ) 3fbf795830

services/search/content.py was a stale copy of src/search/content.py
missing several security improvements:

- _is_private_address did not check addr.is_link_local
- _resolve_hostname_ips did not filter by socket family
- _public_http_url returned True for hostnames with empty DNS
  resolution instead of failing closed
- _get_public_url used bare httpx.get() without connection reuse
- Missing _extract_og_image, thin-content fallback, og_image in result

Replace services/search/content.py with a re-export module that
delegates to the canonical src.search.content implementation.

sleepy referenced this pull request from a commit

2026-06-03 22:24:42 +02:00

[search] Reconcile duplicate search implementations (#771)

sleepy referenced this pull request

2026-06-03 22:25:14 +02:00

[search] Reconcile duplicate search implementations (#771) #856

sleepy closed this pull request

2026-06-04 11:42:21 +02:00