search

rabbitmq / rabbitmq-management

RabbitMQ Management UI and HTTP API
https://www.rabbitmq.com/management.html
Other
370 stars 165 forks source link

API request to aliveness-test when message-ttl is 0 results in a 500 #855

Closed lukebakken closed 3 years ago

lukebakken commented 3 years ago

Reported here:

https://groups.google.com/d/topic/rabbitmq-users/UgXnKBZEaA4/discussion

michaelklishin commented 3 years ago

A policy that sets message TTL to 0 (discards all messages) violates a very fundamental assumption of this check: that published messages can be consumed. I'd say it works as expected but we should respond with a 503 in that case, much like we do in the checks introduced in #854.