Netflix / conductor-community

Apache License 2.0
61 stars 71 forks source link

feature: add option to use ssl on amqp connection #102

Closed rkferreira closed 2 years ago

rkferreira commented 2 years ago

Pull Request type

Changes in this PR

Describe the new behavior from this PR, and why it's needed Issue # event-queue/amqp ssl connection to amqp broker

Alternatives considered

Describe alternative implementation you have considered

useSslProtocol flag implemented

felipesilvamelo28 commented 2 years ago

Very good, this feature is indispensable for us. We've been waiting for her for a long time. Eagerly awaiting approval. Nice work!

rkferreira commented 2 years ago

Hi,

We are using it in production already, would you mind take a look? @apanicker-nflx so we can stop local patching here.

Cheers