Skip to content

use SASL EXTERNAL for RabbitMQ AMQP TLS without credentials#1727

Merged
rickbrouwer merged 1 commit into
kedacore:mainfrom
rickbrouwer:issue-6840
Jun 16, 2026
Merged

use SASL EXTERNAL for RabbitMQ AMQP TLS without credentials#1727
rickbrouwer merged 1 commit into
kedacore:mainfrom
rickbrouwer:issue-6840

Conversation

@rickbrouwer

Copy link
Copy Markdown
Member

Docs for PR kedacore/keda#7606

Checklist

  • Commits are signed with Developer Certificate of Origin (DCO)

@rickbrouwer rickbrouwer requested a review from a team as a code owner April 5, 2026 08:07
@netlify

netlify Bot commented Apr 5, 2026

Copy link
Copy Markdown

Deploy Preview for keda ready!

Name Link
🔨 Latest commit 3147679
🔍 Latest deploy log https://app.netlify.com/projects/keda/deploys/69d21aefb58685000844936e
😎 Deploy Preview https://deploy-preview-1727--keda.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions

github-actions Bot commented Apr 5, 2026

Copy link
Copy Markdown

Thank you for your contribution! 🙏 We will review your PR as soon as possible.

While you are waiting, make sure to:

  • Add your contribution to all applicable KEDA versions
  • GitHub checks are passing
  • Is the DCO check failing? Here is how you can fix DCO issues

Learn more about:

- `key` - Certificate Key for client authentication (optional).

> 💡 **Note:** Using RabbitMQ host with AMQPS protocol will require enabling the TLS settings and passing the required parameters.
> 💡 **Notes:**

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should address this and update the doc accordingly kedacore/keda@e5c66bb#r3053338950

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you make a code proposal @dttung2905 ?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dttung2905 can you check this for me?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @rickbrouwer, I took a look at the docs again and I think we are good to go. Sorry for the confusion

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @dttung2905 !

@rickbrouwer rickbrouwer added keda-pr-already-merged all PR's where the PR's on the KEDA side have already been merged required:keda-v2.20 labels May 7, 2026
@rickbrouwer rickbrouwer requested a review from dttung2905 June 16, 2026 14:52
@rickbrouwer rickbrouwer merged commit cd78676 into kedacore:main Jun 16, 2026
9 checks passed
@rickbrouwer rickbrouwer deleted the issue-6840 branch June 16, 2026 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

keda-pr-already-merged all PR's where the PR's on the KEDA side have already been merged required:keda-v2.20

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants