Skip to content

[codex] fix: validate advertised API version ranges#645

Draft
Herrtian wants to merge 1 commit into
apache:mainfrom
Herrtian:codex/validate-api-version-ranges
Draft

[codex] fix: validate advertised API version ranges#645
Herrtian wants to merge 1 commit into
apache:mainfrom
Herrtian:codex/validate-api-version-ranges

Conversation

@Herrtian

Copy link
Copy Markdown

Summary

  • Validate ApiVersionsResponse api_key, min_version, and max_version values before narrowing them to i16.
  • Return Error::UnsupportedVersion with the offending field name and value instead of panicking on malformed server metadata.
  • Propagate validation failures from the API versions handshake.

Validation

  • cargo fmt --check
  • git diff --check
  • cargo check -p fluss-rs
  • cargo test -p fluss-rs server_api_versions --lib

Closes #535

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Surface a typed error when ApiVersionsResponse fields exceed int16 range

1 participant