njouanin / hbmqtt

MQTT client/broker using Python asynchronous I/O
MIT License
795 stars 188 forks source link

Fix topic filter matching #219

Closed lrasku closed 3 years ago

lrasku commented 3 years ago

Also add two unit tests to test this matching, and fix an incorrect match against + in test_client_subscribe_publish_dollar_topic_2.

This fixes #44.

FlorianLudwig commented 3 years ago

Hi @lrasku

hbmqtt has been deprecated by it's author. We created a fork here: https://github.com/Yakifo/amqtt

I like this change and would like to include it in our fork.

Would you be interested to create your MR there as well?

Thanks :)

lrasku commented 3 years ago

Closing this pull request as this repo is no longer maintained. It has now been merged in Yakifo/amqtt#28.