ryankert01 opened a new pull request, #3025: URL: https://github.com/apache/iggy/pull/3025
## Which issue does this PR close? na ## Rationale The Go SDK defines a `Version` constant but never sends it to the server during login, so the server has no visibility into which SDK version clients are using. ## What changed? `LoginUser.MarshalBinary()` hardcoded an empty string for the version field in the login request. It now sends `contracts.Version` (`"0.7.1-edge.1"`) instead, and the resolved TODO comment in `contracts/version.go` has been removed. ## Local Execution - Passed - Pre-commit hooks passes ## AI Usage 1. Claude Code (Claude Opus 4.6) 2. Help me learn the codebase, and write pr description 3. Verified via `go build`, `go vet`, `go test`, `golangci-lint`, and `typos` — all passing 4. Yes -- 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]
