Skip to content

Conversation

@HsiuChuanHsu
Copy link

Changes

Added support for passwordless authentication mode through a new usePasswordless parameter:

  • Added UsePasswordless field to rabbitMQMetadata struct
  • Added check to require username when usePasswordless=true in Validate() method
  • Added condition to handle passwordless mode in URI parsing in NewRabbitMQScaler()
  • Properly sets Basic Auth header with username only in getQueueInfoViaHTTP()

Checklist

Fixes #6840

@HsiuChuanHsu HsiuChuanHsu requested a review from a team as a code owner October 30, 2025 14:43
@keda-automation keda-automation requested a review from a team October 30, 2025 14:43
@github-actions
Copy link

Thank you for your contribution! 🙏

Please understand that we will do our best to review your PR and give you feedback as soon as possible, but please bear with us if it takes a little longer as expected.

While you are waiting, make sure to:

  • Add an entry in our changelog in alphabetical order and link related issue
  • Update the documentation, if needed
  • Add unit & e2e tests for your changes
  • GitHub checks are passing
  • Is the DCO check failing? Here is how you can fix DCO issues

Once the initial tests are successful, a KEDA member will ensure that the e2e tests are run. Once the e2e tests have been successfully completed, the PR may be merged at a later date. Please be patient.

Learn more about our contribution guide.

@rickbrouwer
Copy link
Member

Wouldn't it be better to create an option in TypedConfig that allows an empty value to be parsed? Is that what you want with this PR? wdyt?

@HsiuChuanHsu HsiuChuanHsu force-pushed the fix/rabbitmq-passwordless-tls-auth branch from fcb1c6c to 89979a2 Compare November 2, 2025 14:00
@HsiuChuanHsu
Copy link
Author

Agree! It would be better to set a option in TypedConfig. Here is the new update summary:

  • Add allowEmpty tag support in typed_config (new constant, Params field, and parsing logic)
  • Mark rabbitmq password field as allowEmpty

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.

Rabbitmq scaler with TLS enabled trigger not working for passwordless authentication

2 participants