MCU8MASS-1828 Use writeCommand instead of writeBytes when closing MQTT session

Author: M65649

examples/provision/provision.ino

@@ -549,12 +549,22 @@ static bool requestAndSaveToNonVolatileMemory(const char* message,
     if (is_certificate) {
         // First erase the existing certifiate at the slot (if any)
         sprintf(command, AT_ERASE_CERTIFICATE, slot);
-        SequansController.writeCommand(command);
+        SequansController.writeBytes(command, strlen(command), true);
+
+        // Dummy read of response, we don't care about the response as the modem
+        // will return an error if there's no certificate to erase and OK if it
+        // was ereased
+        SequansController.readResponse();
 
         sprintf(command, AT_WRITE_CERTIFICATE, slot, data_length);
     } else {
         sprintf(command, AT_ERASE_PRIVATE_KEY, slot);
-        SequansController.writeCommand(command);
+        SequansController.writeBytes(command, strlen(command), true);
+
+        // Dummy read of response, we don't care about the response as the modem
+        // will return an error if there's no certificate to erase and OK if it
+        // was ereased
+        SequansController.readResponse();
 
         sprintf(command, AT_WRITE_PRIVATE_KEY, slot, data_length);
     }

src/mqtt_client.cpp

@@ -409,7 +409,10 @@ bool MqttClientClass::begin(const char* client_id,
     // instantly. We just want to close the current connection if there are any.
     // If there aren't, this will return an error from the modem, but that is
     // fine as it just means that there aren't any connections active.
-    SequansController.readResponse(NULL, 0);
+    //
+    // We do this with writeBytes instead of writeCommand to not issue the
+    // retries of the command if it fails.
+    SequansController.readResponse();
 
     SequansController.clearReceiveBuffer();
 
@@ -540,10 +543,7 @@ bool MqttClientClass::end(void) {
 
     if (isConnected()) {
 
-        SequansController.writeBytes((uint8_t*)MQTT_DISCONNECT,
-                                     strlen(MQTT_DISCONNECT),
-                                     true);
-
+        SequansController.writeCommand(MQTT_DISCONNECT);
         SequansController.clearReceiveBuffer();
 
         connected_to_broker = false;