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

Remove server version validation #1273

Merged
merged 4 commits into from
May 27, 2024
Merged

Commits on May 27, 2024

  1. Protect is_server_compatible against empty version

    Signed-off-by: Tomasz Pietrek <tomasz@nats.io>
    Jarema committed May 27, 2024
    Configuration menu
    Copy the full SHA
    a064353 View commit details
    Browse the repository at this point in the history
  2. Remove check for server compatibility from consumer create

    Checking server version is not a reliable way to know if given
    API call is supported for given context, as client might be connected
    to a leaf node, while the meta leader is a different version, part of
    different cluster.
    
    Signed-off-by: Tomasz Pietrek <tomasz@nats.io>
    Jarema committed May 27, 2024
    Configuration menu
    Copy the full SHA
    5a26c1c View commit details
    Browse the repository at this point in the history
  3. Add test against panic

    When retry on initial connect was used, some JetStream consumer calls
    could panic in specific conditions because server version was not
    available. This test makes sure it will no longer happen.
    
    Signed-off-by: Tomasz Pietrek <tomasz@nats.io>
    Jarema committed May 27, 2024
    Configuration menu
    Copy the full SHA
    606c3e4 View commit details
    Browse the repository at this point in the history
  4. Update dictionary

    Signed-off-by: Tomasz Pietrek <tomasz@nats.io>
    Jarema committed May 27, 2024
    Configuration menu
    Copy the full SHA
    ddb2184 View commit details
    Browse the repository at this point in the history