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

Add jetstream domain to kv-components #2539

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

n-holmstedt
Copy link

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant