Ping response

Wiebe Cazemier
1 parent 76c18f38
Showing 3 changed files with 20 additions and 7 deletions
client.cpp
client.h
mqttpacket.cpp
@@ -2,6 +2,7 @@
 #include <cstring>
 #include <sstream>
+#include <iostream>
 Client::Client(int fd, ThreadData_p threadData) :
     fd(fd),
@@ -65,6 +66,19 @@ void Client::writeMqttPacket(MqttPacket &amp;packet)
     wwi += packet.getSize();
 }
+// Ping responses are always the same, so hardcoding it for optimization.
+void Client::writePingResp()
+{
+    std::cout << "Sending ping response to " << repr() << std::endl;
+
+    if (2 > getWriteBufMaxWriteSize())
+        growWriteBuffer(CLIENT_BUFFER_SIZE);
+
+    writebuf[wwi++] = 0b11010000;
+    writebuf[wwi++] = 0;
+    writeBufIntoFd();
+}
+
 bool Client::writeBufIntoFd() // TODO: ignore the signal BROKEN PIPE we now also get when a client disappears.
 {
     int n;
@@ -92,6 +106,8 @@ bool Client::writeBufIntoFd() // TODO: ignore the signal BROKEN PIPE we now also
     return true;
 }
+
+
 std::string Client::repr()
 {
     std::ostringstream a;
@@ -169,3 +185,5 @@ void Client::setClientProperties(const std::string &amp;clientId, const std::string 
+
+
@@ -90,6 +90,7 @@ public:
     bool getAuthenticated() { return authenticated; }
     bool hasConnectPacketSeen() { return connectPacketSeen; }
+    void writePingResp();
     void writeMqttPacket(MqttPacket &packet);
     bool writeBufIntoFd();
@@ -51,7 +51,7 @@ void MqttPacket::handle()
     if (packetType == PacketType::CONNECT)
         handleConnect();
     else if (packetType == PacketType::PINGREQ)
-        std::cout << "PING" << std::endl;
+        sender->writePingResp();
     else if (packetType == PacketType::SUBSCRIBE)
         handleSubscribe();
 }
@@ -158,12 +158,6 @@ void MqttPacket::handleSubscribe()
     sender->writeBufIntoFd();
 }
-void MqttPacket::handlePing()
-{
-
-}
-
-
 char *MqttPacket::readBytes(size_t length)
 {