Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: skip over KIP-482 tagged fields #2107

Merged
merged 1 commit into from
Jan 12, 2022
Merged

fix: skip over KIP-482 tagged fields #2107

merged 1 commit into from
Jan 12, 2022

Commits on Jan 11, 2022

  1. fix: skip over KIP-482 tagged fields

    The decoder would error out with "non-empty tagged fields are not
    supported yet" if it encountered a non-zero length for the tagged fields
    array. Handle these more graciously (as per the KIP) by skipping over
    them without deserializing.
    
    This fixes an issue with the v3 ApiVersionsResponse on Kafka 2.7.0 and
    newer which now return the `SupportedFeatures` tagged field on the
    response (KAFKA-10028).
    
    Signed-off-by: Dominic Evans <dominic.evans@uk.ibm.com>
    dnwe committed Jan 11, 2022
    Configuration menu
    Copy the full SHA
    31a5366 View commit details
    Browse the repository at this point in the history