seokjin0414 opened a new pull request, #2768: URL: https://github.com/apache/iggy/pull/2768
## Summary Closes #2729 - Allow hostnames (e.g., `localhost:8080`) in QUIC and WebSocket client server addresses by using `tokio::net::lookup_host()` for DNS resolution - QUIC `create()`: defer `SocketAddr` parsing, store raw server address string (TCP pattern) - QUIC `connect()`: replace `parse::<SocketAddr>()` with `lookup_host()` for hostname resolution - WebSocket `connect()`: replace `parse::<SocketAddr>()` with `lookup_host()` for hostname resolution - Remove `localhost` → `127.0.0.1` workaround in CLI args since SDK now handles hostnames natively - Add hostname example to iggy-bench documentation ## Test plan - [x] QUIC: `create()` with hostname (`localhost:8080`) succeeds - [x] QUIC: `create()` with FQDN (`my-server.example.com:8080`) succeeds - [x] QUIC: raw hostname stored in `current_server_address` - [x] QUIC: `from_connection_string` with hostname succeeds - [x] WebSocket: `from_connection_string` with hostname succeeds - [x] All existing tests pass (iggy: 100, iggy_common: 333) - [x] `cargo clippy` clean - [x] `cargo fmt` clean -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
