feat: update -ldp option to show default ports in CLI output

[ehsandeep]

• Modified URL formatting in runner.go to respect LeaveDefaultPorts option
• Fixed AddURLDefaultPort function to actually add default ports (80/443)
• When -ldp is used, CLI output now shows https://example.com:443 instead of https://example.com
• Maintains backward compatibility - default behavior unchanged

Fixes CLI output inconsistency where -ldp flag only affected Host headers but not the displayed URL in console output.

Closes #2328

~/Github/httpx $ echo example.com | ./httpx -ldp -p http:80 -silent
http://example.com:80

~/Github/httpx $ echo example.com | ./httpx -ldp -p https:443 -silent
https://example.com:443

~/Github/httpx $ echo example.com | ./httpx -ldp -silent
https://example.com:443

Summary by CodeRabbit

Release Notes

• New Features

  • Introduced a new setting to control default port display behavior in URLs, allowing default ports (80 for HTTP, 443 for HTTPS) to be shown or hidden in output and logs as needed.

• Bug Fixes

  • Improved URL formatting logic to consistently apply default ports to URLs lacking explicit port specifications.

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

[coderabbitai]

Walkthrough

This change implements functionality to conditionally include default ports (80 for HTTP, 443 for HTTPS) in URL output. The AddURLDefaultPort function in stringz.go now explicitly appends default ports to URLs that lack them, while runner.go conditionally applies this behavior based on a LeaveDefaultPorts flag to control output formatting.

Changes

Cohort / File(s)	Summary
URL Port Handling common/stringz/stringz.go, runner/runner.go	stringz.go: Adds logic to AddURLDefaultPort to explicitly append default port (80 for HTTP, 443 for HTTPS) when URL lacks a port by checking and updating parsed URL's Port field. runner.go: Replaces unconditional default port removal with conditional logic—when LeaveDefaultPorts is true, adds back default port; otherwise removes it as before.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

• Verify LeaveDefaultPorts flag is properly initialized, passed through the execution flow, and accessible where needed in runner.go
• Confirm port-appending logic in stringz.go handles edge cases (malformed URLs, already-present ports, scheme detection accuracy)
• Check that both HTTP (80) and HTTPS (443) port defaults are correctly identified and applied consistently
• Validate downstream impact on output formatting and any tools consuming the URL strings

Poem

🐰 Ports were hiding, now they show,
Eighty, four-four-three in tow!
With a flag to lead the way,
Default ports see the light of day! ✨

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and concisely describes the main change: updating the -ldp option to show default ports in CLI output, which matches the primary code modifications.
Linked Issues check	✅ Passed	The code changes implement all primary requirements: the -ldp flag now shows default ports (80/443) in CLI output as requested in issue #2328.
Out of Scope Changes check	✅ Passed	All changes are directly related to the linked issue objectives—modifications to AddURLDefaultPort and runner.go to implement the -ldp flag functionality for displaying default ports.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

✨ 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 feature/ldp-cli-output-ports

---

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