search

amqp / rhea

A reactive messaging library based on the AMQP protocol
Apache License 2.0
273 stars 80 forks source link

Check that frame.payload is truthy #389

Closed GeorgeJCleary closed 1 year ago

GeorgeJCleary commented 1 year ago

Fix for regression issue for https://github.com/amqp/rhea/issues/388

grs commented 1 year ago

Thank you @GeorgeJCleary !

RMueAZD commented 1 year ago

@GeorgeJCleary Thanks!

I'm still confused as to why this empty frame is being sent in the first place.

Plus (this kept me from creating a PR myself): I couldn't figure out how to create test coverage for this to prevent future regressions.