Add Message ctor and use make_unique

Use topic available at subscribe time instead of receive time to better handle wildcards

Signed-off-by: Kai-Uwe Hermann <kai-uwe.hermann@pionix.de>

Author: hikinggrass

lib/everest/framework/include/utils/message_queue.hpp

@@ -22,6 +22,8 @@ namespace Everest {
 struct Message {
     std::string topic;   ///< The MQTT topic where this message originated from
     std::string payload; ///< The message payload
+
+    Message(const std::string& topic_, const std::string& payload_) : topic(topic_), payload(payload_) {}
 };
 
 struct ParsedMessage {

lib/everest/framework/lib/mqtt_abstraction_impl.cpp

@@ -203,9 +203,9 @@ void MQTTAbstractionImpl::subscribe(const std::string& topic, QOS qos) {
 
     const auto result = this->mqtt_client->subscribe(
         topic,
-        [this](everest::lib::io::mqtt::mosquitto_cpp& client,
+        [this, topic](everest::lib::io::mqtt::mosquitto_cpp& client,
                everest::lib::io::mqtt::mosquitto_cpp::message const& message) {
-            this->message_queue.add(std::unique_ptr<Message>(new Message{message.topic, message.payload}));
+            this->message_queue.add(std::make_unique<Message>(topic, message.payload));
         },
         max_qos_level);
 }