Skip to content

Conversation

@the-mikedavis
Copy link
Collaborator

The protocol-listener health check accepts protocol names with 's' for the encrypted listeners like amqps and mqtts. The health check distinguishes between encrypted and non-encrypted so to get a 200 when running AMQPS but not AMQP you need to query for amqps.

This change mentions the encrypted protocol names. I've also combined amqp091 and amqp10 as amqp which is what the health check translates these into.

Also see https://github.com/rabbitmq/rabbitmq-server/blob/c7f6cad331443df0eaad668c2f1ca62410259c3a/deps/rabbitmq_management/src/rabbit_mgmt_wm_health_check_protocol_listener.erl#L71-L135. The health check normalizes the protocol names so it can accept quite a large range

The protocol-listener health check accepts protocol names with 's' for
the encrypted listeners like `amqps` and `mqtts`. The health check
distinguishes between encrypted and non-encrypted so to get a 200 when
running AMQPS but not AMQP you need to query for `amqps`.

This change mentions the encrypted protocol names. I've also combined
`amqp091` and `amqp10` as `amqp` which is what the health check
translates these into.
@the-mikedavis the-mikedavis self-assigned this Aug 21, 2025
@the-mikedavis
Copy link
Collaborator Author

I applied the change to the 4.0 docs as well since even before rabbitmq/rabbitmq-server#13871 (not backported to 4.0) the check accepted amqps, mqtts, etc - that stuff didn't change in rabbitmq/rabbitmq-server#13871.

@cloudflare-workers-and-pages
Copy link

Deploying rabbitmq-website with  Cloudflare Pages  Cloudflare Pages

Latest commit: 19a2705
Status: ✅  Deploy successful!
Preview URL: https://ac65c132.rabbitmq-website.pages.dev
Branch Preview URL: https://md-protocol-listener-s.rabbitmq-website.pages.dev

View logs

@michaelklishin michaelklishin added this to the 4.2.0 milestone Aug 21, 2025
@michaelklishin
Copy link
Contributor

@the-mikedavis thank you!

@michaelklishin michaelklishin merged commit fe8f441 into main Aug 21, 2025
1 check passed
@michaelklishin michaelklishin deleted the md/protocol-listener-s branch August 21, 2025 22:42
@the-mikedavis the-mikedavis restored the md/protocol-listener-s branch August 22, 2025 03:50
@the-mikedavis the-mikedavis deleted the md/protocol-listener-s branch August 22, 2025 03:50
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.

3 participants