Skip to content

[pull] master from kevoreilly:master#460

Merged
pull[bot] merged 4 commits intothreatcode:masterfrom
kevoreilly:master
Apr 30, 2026
Merged

[pull] master from kevoreilly:master#460
pull[bot] merged 4 commits intothreatcode:masterfrom
kevoreilly:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull Bot commented Apr 30, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

wmetcalf and others added 4 commits April 14, 2026 18:35
Add optional DoH support to encrypt all post-analysis DNS lookups
(forward and reverse) via HTTPS instead of plaintext UDP/TCP DNS.

When enabled, all calls to resolve() and reverse DNS in _enrich_hosts()
route through a configurable DoH provider (default: Google). This
prevents DNS query leakage from the analysis host.

Configuration in cuckoo.conf [processing] section (same as reverse_dns):
  dns_over_https = yes
  doh_url = https://dns.google/resolve

Compatible with any provider supporting application/dns-json:
- Google: https://dns.google/resolve (default)
- Cloudflare: https://cloudflare-dns.com/dns-query
- Custom/self-hosted DoH resolvers

Changes:
- lib/cuckoo/common/dns.py: Add resolve_doh(), set_doh(), set_doh_url()
  with requests.Session for connection pooling, rdtype-aware response
  filtering, HTTPS URL validation, and specific exception handling
- modules/processing/network.py: Read config, use DoH for PTR lookups,
  normalize trailing dots in both DoH and non-DoH PTR paths
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
feat: add DNS-over-HTTPS support for post-analysis hostname resolution
@pull pull Bot locked and limited conversation to collaborators Apr 30, 2026
@pull pull Bot added the ⤵️ pull label Apr 30, 2026
@pull pull Bot merged commit f0d5e86 into threatcode:master Apr 30, 2026
3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants