fix: host JSON field now returns hostname instead of IP

[ehsandeep]

• Changed 'host' field to return actual hostname (e.g., example.com)
• Added new 'host_ip' field for the resolved/dialed IP address
• Fixes semantic issue where 'host' was incorrectly returning IP

Closes #2332

$ echo www.example.com | ./httpx -silent  -j | jq .
{
  "timestamp": "2025-12-06T22:22:11.061425+05:30",
  "port": "443",
  "url": "https://www.example.com",
  "input": "www.example.com",
  "title": "Example Domain",
  "scheme": "https",
  "content_type": "text/html",
  "method": "GET",
  "host": "www.example.com",
  "host_ip": "2600:140f:5::173f:6cf3",
  "path": "/",
  "time": "97.401875ms",
  "a": [
    "23.54.80.226",
    "23.54.80.218"
  ],
  "aaaa": [
    "2600:140f:5::173f:6cda",
    "2600:140f:5::173f:6cf3"
  ],
  "cname": [
    "www.example.com-v4.edgesuite.net",
    "a1422.dscr.akamai.net"
  ],
  "tech": [
    "HTTP/3"
  ],
  "words": 25,
  "lines": 1,
  "status_code": 200,
  "content_length": 513,
  "failed": false,
  "knowledgebase": {
    "PageType": "nonerror",
    "pHash": 0
  },
  "resolvers": [
    "100.95.0.252:53",
    "100.95.0.253:53"
  ]
}

Summary by CodeRabbit

• New Features
  • Separated hostname and IP address information in results for improved clarity and organization.
  • Host field now displays the hostname instead of IP address, with IP address available in a dedicated field.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

The changes modify the Result struct to separate hostname and resolved IP address. The host field now returns the actual hostname parsed from the URL, while a new host_ip field stores the resolved IP address that was previously conflated in the host field.

Changes

Cohort / File(s)	Summary
Result struct enhancement runner/types.go, runner/runner.go	Added HostIP field to Result struct to store resolved IP address separately from hostname. Modified Host field assignment to use parsed URL hostname instead of IP address.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

• Straightforward data structure modification affecting only 2 files
• Field addition with standard struct tags (JSON/CSV/MapStructure)
• Simple reassignment of Host field source with new HostIP population
• Low logic density; verify correct tag naming consistency and that HostIP is properly populated at the assignment site

Poem

🐰 A hop, skip, and a field away,
Host now shows its proper name today!
No more IPs where hostnames should be—
HostIP joins the party, wild and free!
The rabbit celebrates with glee! 🎉

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: the host JSON field now returns hostname instead of IP.
Linked Issues check	✅ Passed	The changes address all requirements from #2332: host field returns hostname from URL and new host_ip field stores resolved IP address.
Out of Scope Changes check	✅ Passed	All changes are scoped to the linked issue requirements: hostname/IP field corrections in runner/runner.go and runner/types.go.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix-host-json-field

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8b04cd9 and 97ddb0b.

📒 Files selected for processing (2)

• runner/runner.go (1 hunks)
• runner/types.go (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (6)

• GitHub Check: Functional Test (macOS-latest)
• GitHub Check: Lint Test
• GitHub Check: Functional Test (ubuntu-latest)
• GitHub Check: Functional Test (windows-latest)
• GitHub Check: release-test
• GitHub Check: Analyze (go)

🔇 Additional comments (2)

runner/types.go (1)

62-62: LGTM! Field definition follows struct conventions.

The new HostIP field is correctly defined with appropriate JSON, CSV, and MapStructure tags. The placement next to the existing Host field (line 61) is logical and maintains good code organization. The field will be properly serialized in all output formats and available as a DSL variable for filtering/matching expressions.

runner/runner.go (1)

2329-2330: LGTM! Host/HostIP separation correctly implemented.

The changes properly address the semantic issue by separating hostname from IP address. The Host field contains the hostname parsed from the URL via parsed.Hostname() (without port), while HostIP stores the actual IP address connected to. This distinction handles multiple scenarios correctly:

• Direct IP input: both fields contain the same IP
• Hostname with port: Host is hostname only, HostIP is the resolved/connected IP
• CustomIP targets: Host preserves the original hostname, HostIP contains the custom IP
• DNS failures: Host contains the hostname, HostIP may be empty (gracefully handled downstream)

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}