-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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 support for credentials file to nats_consumer and nats output #7022
Conversation
Thanks @ripienaar, interested to see the new features, just go ahead and add your changes on top of this changeset and we can merge them in one go. |
@danielnelson OK am about done here, mainly I added Additionally moved away from the now deprecated method of connecting via opts. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. I was unsure from your last comment, was there anything else you wanted to modify first? The only thing I noticed was that we need to get to update nats_consumer README as well.
plugins/outputs/nats/README.md
Outdated
# credentials = "/etc/telegraf/nats.creds | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add this example to the nats_consumer
README?
I've updated the comments etc, it's ready to merge if you agree |
I would like to make some follow up PRs to support some of the modern NATS 2.0 security
features and thought I'd start by updating dependencies first before doing those changes
Required for all PRs: