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

DBZ-7829 Added nats JWT/seed authentication config options. #82

Closed
wants to merge 2 commits into from

Conversation

calmera
Copy link
Contributor

@calmera calmera commented Mar 19, 2024

Added two new properties; auth.jwt and auth.seed to allow to pass a JWT and a seed to connect to nats when it has been configured for decentralized auth (like Synadia)

Added two new properties; `auth.jwt` and `auth.seed` to allow to pass a
JWT and a seed to connect to nats when it has been configured for
decentralized auth (like Synadia)

Signed-off-by: Daan Gerits <[email protected]>
@jpechane
Copy link
Contributor

@calmera Hi, thanks for the PR. Could you please create a Jira describing the new feature, modify the commit message to be DBZ-xxxx Added nats JWT/seed authentication config options where xxxx is the correct number.
Also use mvn clean install to make sure the code is correctly formatted before pushing.

Signed-off-by: Daan Gerits <[email protected]>
@calmera calmera changed the title Added nats JWT/seed authentication config options. DBZ-7829 Added nats JWT/seed authentication config options. Apr 29, 2024
@calmera
Copy link
Contributor Author

calmera commented Apr 29, 2024

@jpechane Should be done. Let me know if there is anything else for me to look at.

@jpechane
Copy link
Contributor

Thanks a lot!
Closing in favour of #86 that contains reformatted text and merged commits. I'll merge the PR as soon as CI is finished.
Could you please also prepare a docs PR for the new feature?

@jpechane jpechane closed this Apr 29, 2024
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.

2 participants