Add jetstream domain to kv-components #2539
Open
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I noticed that the nats components using jetstream is missing the ability to specify jetstream-domain allowing a client to use jetstream-enabled leaf nodes. However, I opted for changing the key-value store components since these already use the new jetstream api and the old ones prob. needs to be rewritten at some point anyway.
I also took the liberty to add username and password authentication to the nats auth section. This isn't really something one would want to use in production, but helps with testing leafnodes connectivity since they need to be bound with an account and connects user credentials (preferably username/password when testing locally).
This is my first PR for Benthos so i'm not super-familiar with the structure. Things seemed to fit, runs tests/compiles, and works when i try it locally so hopefully it isn't that bad. Some integration-tests failed, but that seems to have nothing to do with the changes i made.