Closed ckreiling closed 1 year ago
Follow-up to https://github.com/batchcorp/plumber-schemas/pull/87
Related to https://github.com/batchcorp/plumber/issues/335. In addition this PR adds a few other options I'd like to be able to set.
To my knowledge, Pulsar doesn't support multiple forms of authentication, so I added a validation to raise an error in the event TLS auth and token auth are both set. I figured it's easier to remove that validation than add it in the future
cc @blinktag
Awesome thank you @ckreiling We'll get this released shortly
My pleasure! Thanks for the speedy replies. Let me know if I need to / how to fix the coverage upload step that failed
Follow-up to https://github.com/batchcorp/plumber-schemas/pull/87
Related to https://github.com/batchcorp/plumber/issues/335. In addition this PR adds a few other options I'd like to be able to set.
To my knowledge, Pulsar doesn't support multiple forms of authentication, so I added a validation to raise an error in the event TLS auth and token auth are both set. I figured it's easier to remove that validation than add it in the future
cc @blinktag