lucadentella
diff --git a/‎cpp_utils/BLERemoteService.cpp
Lines changed: 12 additions & 1 deletion b/‎cpp_utils/BLERemoteService.cpp
Lines changed: 12 additions & 1 deletion
diff --git a/‎cpp_utils/BLEUtils.cpp
Lines changed: 36 additions & 3 deletions b/‎cpp_utils/BLEUtils.cpp
Lines changed: 36 additions & 3 deletions
diff --git a/‎cpp_utils/tests/test_rest.cpp
Lines changed: 5 additions & 0 deletions b/‎cpp_utils/tests/test_rest.cpp
Lines changed: 5 additions & 0 deletions
@@ -154,6 +154,7 @@ BLERemoteCharacteristic* BLERemoteService::getCharacteristic(BLEUUID uuid) {
 
 /**
  * @brief Retrieve all the characteristics for this service.
+ * This function will not return until we have all the characteristics.
  * @return N/A
  */
 void BLERemoteService::retrieveCharacteristics() {
@@ -256,7 +257,8 @@ void BLERemoteService::retrieveCharacteristics() {
 		m_characteristicMap.insert(std::pair<std::string, BLERemoteCharacteristic*>(pNewRemoteCharacteristic->getUUID().toString(), pNewRemoteCharacteristic));
 
 		offset++;   // Increment our count of number of descriptors found.
-	}
+	} // Loop forever (until we break inside the loop).
+
 	m_haveCharacteristics = true; // Remember that we have received the characteristics.
 	ESP_LOGD(LOG_TAG, "<< getCharacteristics()");
 } // getCharacteristics
@@ -267,6 +269,12 @@ void BLERemoteService::retrieveCharacteristics() {
  * @return A map of all the characteristics of this service.
  */
 std::map<std::string, BLERemoteCharacteristic*>* BLERemoteService::getCharacteristics() {
+	// If is possible that we have not read the characteristics associated with the service so do that
+	// now.  The request to retrieve the characteristics by calling "retrieveCharacteristics" is a blocking
+	// call and does not return until all the characteristics are available.
+	if (!m_haveCharacteristics) {
+		retrieveCharacteristics();
+	}
 	return &m_characteristicMap;
 } // getCharacteristics
 
@@ -275,14 +283,17 @@ BLEClient* BLERemoteService::getClient() {
 	return m_pClient;
 }
 
+
 uint16_t BLERemoteService::getEndHandle() {
 	return m_endHandle;
 }
 
+
 esp_gatt_id_t* BLERemoteService::getSrvcId() {
 	return &m_srvcId;
 }
 
+
 uint16_t BLERemoteService::getStartHandle() {
 	return m_startHandle;
 }
 
@@ -959,8 +959,8 @@ const char* BLEUtils::devTypeToString(esp_bt_dev_type_t type) {
  * @brief Dump the GAP event to the log.
  */
 void BLEUtils::dumpGapEvent(
-	esp_gap_ble_cb_event_t event,
-	esp_ble_gap_cb_param_t *param) {
+	esp_gap_ble_cb_event_t  event,
+	esp_ble_gap_cb_param_t* param) {
 	ESP_LOGD(LOG_TAG, "Received a GAP event: %s", gapEventToString(event));
 	switch(event) {
 		//
@@ -1091,14 +1091,19 @@ void BLEUtils::dumpGapEvent(
 			break;
 		} // ESP_GAP_BLE_READ_RSSI_COMPLETE_EVT
 
+
 		//
 		// ESP_GAP_BLE_SCAN_PARAM_SET_COMPLETE_EVT
 		//
+		// scan_param_cmpl.
+		// - esp_bt_status_t status
+		//
 		case ESP_GAP_BLE_SCAN_PARAM_SET_COMPLETE_EVT: {
 			ESP_LOGD(LOG_TAG, "[status: %d]",	param->scan_param_cmpl.status);
 			break;
 		} // ESP_GAP_BLE_SCAN_PARAM_SET_COMPLETE_EVT
 
+
 		//
 		// ESP_GAP_BLE_SCAN_RESULT_EVT
 		//
@@ -1112,6 +1117,7 @@ void BLEUtils::dumpGapEvent(
 		// - ble_adv
 		// - flag
 		// - num_resps
+		//
 		case ESP_GAP_BLE_SCAN_RESULT_EVT: {
 			switch(param->scan_rst.search_evt) {
 				case ESP_GAP_SEARCH_INQ_RES_EVT: {
@@ -1139,9 +1145,23 @@ void BLEUtils::dumpGapEvent(
 		} // ESP_GAP_BLE_SCAN_RESULT_EVT
 
 
+		//
+		// ESP_GAP_BLE_SCAN_RSP_DATA_SET_COMPLETE_EVT
+		//
+		// scan_rsp_data_cmpl
+		// - esp_bt_status_t status
+		//
+		case ESP_GAP_BLE_SCAN_RSP_DATA_SET_COMPLETE_EVT: {
+			ESP_LOGD(LOG_TAG, "[status: %d]",	param->scan_rsp_data_cmpl.status);
+			break;
+		} // ESP_GAP_BLE_SCAN_RSP_DATA_SET_COMPLETE_EVT
+
+
 		//
 		// ESP_GAP_BLE_SCAN_START_COMPLETE_EVT
 		//
+		// scan_start_cmpl
+		// - esp_bt_status_t status
 		case ESP_GAP_BLE_SCAN_START_COMPLETE_EVT: {
 			ESP_LOGD(LOG_TAG, "[status: %d]", param->scan_start_cmpl.status);
 			break;
@@ -1151,14 +1171,26 @@ void BLEUtils::dumpGapEvent(
 		//
 		// ESP_GAP_BLE_SCAN_STOP_COMPLETE_EVT
 		//
+		// scan_stop_cmpl
+		// - esp_bt_status_t status
+		//
 		case ESP_GAP_BLE_SCAN_STOP_COMPLETE_EVT: {
 			ESP_LOGD(LOG_TAG, "[status: %d]", param->scan_stop_cmpl.status);
 			break;
 		} // ESP_GAP_BLE_SCAN_STOP_COMPLETE_EVT
 
 
 		//
-		// ESP_GAP_BLE_SCAN_UPDATE_CONN_PARAMS_EVT
+		// ESP_GAP_BLE_UPDATE_CONN_PARAMS_EVT
+		//
+		// update_conn_params
+		// - esp_bt_status_t status
+		// - esp_bd_addr_t bda
+		// - uint16_t min_int
+		// - uint16_t max_int
+		// - uint16_t latency
+		// - uint16_t conn_int
+		// - uint16_t timeout
 		//
 		case ESP_GAP_BLE_UPDATE_CONN_PARAMS_EVT: {
 			ESP_LOGD(LOG_TAG, "[status: %d, bd_addr: %s, min_int: %d, max_int: %d, latency: %d, conn_int: %d, timeout: %d]",
@@ -1182,6 +1214,7 @@ void BLEUtils::dumpGapEvent(
 			break;
 		} // ESP_GAP_BLE_SEC_REQ_EVT
 
+
 		default: {
 			ESP_LOGD(LOG_TAG, "*** dumpGapEvent: Logger not coded ***");
 			break;
 
@@ -1,5 +1,10 @@
 /*
  * Test the REST API client.
+ * This application leverages LibCurl.  You must make that package available
+ * as well as "enable it" from "make menuconfig" and C++ Settings -> libCurl present.
+ * See also:
+ * * https://github.com/nkolban/esp32-snippets/issues/108
+ *
  */
 #include <curl/curl.h>
 #include <esp_log.h>