maruina / go-infrabin / 18815534996

56%

push

github

Add /egress endpoint (#410)

* Add egress endpoints for network connectivity testing

Implements DNS resolution, HTTP, and HTTPS connectivity testing endpoints
with support for custom DNS servers and configurable timeouts.

Closes #49

* Add custom DNS server support to EgressDNS endpoint

Allow users to specify a custom DNS server for DNS resolution testing
with format hostname@dns_server:port. Port defaults to 53 if not specified.

Changes:
- Updated proto definition to document custom DNS format
- Modified EgressDNS implementation to parse and use custom DNS servers
- Added DNS server address validation (host:port format)
- Added 3 new test cases for custom DNS functionality
- Updated README with examples of custom DNS usage

Examples:
- google.com@8.8.8.8:53 (uses Google DNS with explicit port)
- google.com@1.1.1.1 (uses Cloudflare DNS, port defaults to 53)
- google.com (uses system DNS when @ is omitted)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

* Refactor EgressDNS to minimize if/else nesting

Extract DNS resolver creation logic into a separate helper method
to reduce nested if statements and improve code readability.

Changes:
- Created createDNSResolver helper method
- Reduced if/else nesting from 3 levels to 1 level
- Early return pattern for empty dnsServer
- Clearer separation of concerns
- All tests still passing

Benefits:
- Improved readability and maintainability
- Follows Go best practices for minimal nesting
- Easier to test resolver creation logic independently

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

* Fix errcheck linter: explicitly ignore resp.Body.Close() error

The golangci-lint errcheck linter requires that all error return
values are checked. Since we're closing the response body in a defer
after successfully receiving and processing the response, any close
error at this point is not ac... (continued)

130 of 168 new or added lines in 3 files covered. (77.38%)

482 of 894 relevant lines covered (53.91%)

4.38 hits per line