chore: Update error path for Notecard v7.4.1

Author: zfields

src/Arduino_NotecardConnectionHandler.cpp

@@ -661,15 +661,8 @@ bool NotecardConnectionHandler::armInterrupt (void) const
         // Check the response for errors
         if (NoteResponseError(rsp)) {
           const char *err = JGetString(rsp, "err");
-          // This error must be ignored. As of LTSv6, `rearm` is not idempotent.
-          // For now, we are counting on the fact that it is highly unlikely any
-          // severe errors would occur in isolation. Once the Notecard firmware
-          // is updated to support idempotent `rearm` requests, this error will
-          // be handled as a failure.
-          Debug.print(DBG_VERBOSE, F("%s"), err);
-          result = true;  // Ignore the error
-          // Debug.print(DBG_ERROR, F("%s\n"), err);
-          // result = false;
+          Debug.print(DBG_ERROR, F("%s\n"), err);
+          result = false;
         } else {
           result = true;
         }
@@ -868,8 +861,8 @@ bool NotecardConnectionHandler::updateUidCache (void)
 #endif
 
   // This operation is safe to perform before a sync has occurred, because the
-  // Notecard UID is static and the cloud value of Serial Number is ultimately
-  // given preference to the value reported by the device.
+  // Notecard UID is static and the cloud value of Serial Number is strictly
+  // informational with regard to the host firmware operations.
 
   // Read the Notecard UID from the Notehub configuration
   if (J *rsp = _notecard.requestAndResponse(_notecard.newRequest("hub.get"))) {