search

gftea / amqprs

Async & Lock-free RabbitMQ Rust Client, Easy-to-use API
MIT License
214 stars 27 forks source link

ConsumerMessage.delivery and .basic_properties should not be options #89

Closed michaelklishin closed 1 year ago

michaelklishin commented 1 year ago

A message can carry no payload but it will always have delivery metadata (e.g. delivery tag) and its own metadata (basic properties). So it does not make sense to use Option<T> for them.

gftea commented 1 year ago

Yes, see #68

michaelklishin commented 1 year ago

OK, I'll assume this is a duplicate.