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.

Sign up for GitHub

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

Jump to bottom

Fix serde Stream roundtrip #1294

Merged
merged 2 commits into from
Aug 5, 2024
Merged

Fix serde Stream roundtrip #1294

merged 2 commits into from
Aug 5, 2024

Commits on Aug 2, 2024

  1. Fix serde Stream roundtrip 

    If someone does a serde serialize/deserialize roundtrip with
jetstream::stream::Stream, it will result in error, as then
`ConsumerLimits` will be `null`.
This should not happen in real-world scenarios, as server does not send
nulls for those values, however its good to protect against those
scenarios in case Go/nats-server changes its behaviour as some point in time.

Signed-off-by: Tomasz Pietrek <[email protected]>
    @Jarema
    Jarema committed Aug 2, 2024
    Configuration menu
    Copy the full SHA
    5c26287 View commit details
    Browse the repository at this point in the history

Commits on Aug 5, 2024

  1. Fix lint 

    Signed-off-by: Tomasz Pietrek <[email protected]>
    @Jarema
    Jarema committed Aug 5, 2024
    Configuration menu
    Copy the full SHA
    29f76e8 View commit details
    Browse the repository at this point in the history