Skip to content

Commit

Permalink
fix(store): encode nats-js-kv keys
Browse files Browse the repository at this point in the history
Signed-off-by: jkoberg <[email protected]>
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
  • Loading branch information
kobergj authored and butonic committed Sep 11, 2024
1 parent 2e11551 commit dc5a8dd
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 0 deletions.
6 changes: 6 additions & 0 deletions changelog/unreleased/fix-nats-encoding.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
Bugfix: Fix nats encoding

Encode nats-js-kv keys. This got lost by a dependency bump.

https://github.com/cs3org/reva/pull/4842
https://github.com/cs3org/reva/pull/4678
1 change: 1 addition & 0 deletions pkg/store/store.go
Original file line number Diff line number Diff line change
Expand Up @@ -152,6 +152,7 @@ func Create(opts ...microstore.Option) microstore.Store {
return natsjskv.NewStore(
append(opts,
natsjskv.NatsOptions(natsOptions), // always pass in properly initialized default nats options
natsjskv.EncodeKeys(),

Check failure on line 155 in pkg/store/store.go

View workflow job for this annotation

GitHub Actions / lint

undefined: natsjskv.EncodeKeys (typecheck)

Check failure on line 155 in pkg/store/store.go

View workflow job for this annotation

GitHub Actions / lint

undefined: natsjskv.EncodeKeys) (typecheck)

Check failure on line 155 in pkg/store/store.go

View workflow job for this annotation

GitHub Actions / lint

undefined: natsjskv.EncodeKeys) (typecheck)

Check failure on line 155 in pkg/store/store.go

View workflow job for this annotation

GitHub Actions / lint

undefined: natsjskv.EncodeKeys) (typecheck)
natsjskv.DefaultTTL(ttl))...,
)
case TypeMemory, "mem", "": // allow existing short form and use as default
Expand Down

0 comments on commit dc5a8dd

Please sign in to comment.