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(kv): Fixes key listing for async client to match go client #792

Merged
merged 1 commit into from
Jan 20, 2023
Merged

fix(kv): Fixes key listing for async client to match go client #792

merged 1 commit into from
Jan 20, 2023

Commits on Jan 19, 2023

  1. fix(kv): Fixes key listing for async client to match go client 

    The go client [filters out deleted and purged keys](https://github.com/nats-io/nats.go/blob/66009489432be03ff32e5384a2b502fe88f4c635/kv.go#L909)
and the async client was not filtering out those keys. This fixes the
behavior to be the same and is also slightly more efficient as we aren't iterating
over the _entire_ stream of key value history to get the current keys

Signed-off-by: Taylor Thomas <taylor@cosmonic.com>
    @thomastaylor312
    thomastaylor312 committed Jan 19, 2023
    Configuration menu
    Copy the full SHA
    e969343 View commit details
    Browse the repository at this point in the history