Skip to content

Commit b82f040

Browse files
committed
applications, samples: remove auth status log when no pairing support
When pairing is not supported, the authentication status conveyed by the BLE_GAP_EVT_AUTH_STATUS event is of little value to the application. Remove the auth status logging in samples and apps that signals to the SoftDevice that pairing is not supported. Signed-off-by: Andreas Moltumyr <andreas.moltumyr@nordicsemi.no>
1 parent 0d61c5a commit b82f040

6 files changed

Lines changed: 8 additions & 29 deletions

File tree

applications/firmware_loader/ble_mcumgr/src/main.c

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -80,13 +80,6 @@ static void on_ble_evt(const ble_evt_t *evt, void *ctx)
8080
break;
8181
}
8282

83-
case BLE_GAP_EVT_AUTH_STATUS:
84-
{
85-
LOG_INF("Authentication status %#x",
86-
evt->evt.gap_evt.params.auth_status.auth_status);
87-
break;
88-
}
89-
9083
case BLE_GAP_EVT_SEC_PARAMS_REQUEST:
9184
{
9285
/* Pairing not supported */

doc/nrf-bm/release_notes/release_notes_changelog.rst

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -117,6 +117,14 @@ Bluetooth LE samples
117117
* :ref:`ble_nus_sample`
118118
* :ref:`ble_mcuboot_recovery_entry_sample`
119119

120+
* Removed the authentication status logging from the following samples and applications that do not support pairing (do not use the :ref:`lib_peer_manager` library):
121+
122+
* :ref:`ug_dfu_firmware_loader` (Bluetooth LE)
123+
* :ref:`ble_lbs_sample`
124+
* :ref:`ble_nus_sample`
125+
* :ref:`ble_pwr_profiling_sample`
126+
* :ref:`ble_mcuboot_recovery_entry_sample`
127+
120128
* :ref:`ble_cgms_sample` sample:
121129

122130
* Removed the call to the :c:func:`sd_ble_gatts_sys_attr_set` function from the main source file.

samples/bluetooth/ble_lbs/src/main.c

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -48,11 +48,6 @@ static void on_ble_evt(const ble_evt_t *evt, void *ctx)
4848
nrf_gpio_pin_write(BOARD_PIN_LED_1, !BOARD_LED_ACTIVE_STATE);
4949
break;
5050

51-
case BLE_GAP_EVT_AUTH_STATUS:
52-
LOG_INF("Authentication status: %#x",
53-
evt->evt.gap_evt.params.auth_status.auth_status);
54-
break;
55-
5651
case BLE_GAP_EVT_SEC_PARAMS_REQUEST:
5752
/* Pairing not supported */
5853
nrf_err = sd_ble_gap_sec_params_reply(evt->evt.gap_evt.conn_handle,

samples/bluetooth/ble_nus/src/main.c

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -225,11 +225,6 @@ static void on_ble_evt(const ble_evt_t *evt, void *ctx)
225225

226226
break;
227227

228-
case BLE_GAP_EVT_AUTH_STATUS:
229-
LOG_INF("Authentication status: %#x",
230-
evt->evt.gap_evt.params.auth_status.auth_status);
231-
break;
232-
233228
case BLE_GAP_EVT_SEC_PARAMS_REQUEST:
234229
/* Pairing not supported */
235230
nrf_err = sd_ble_gap_sec_params_reply(evt->evt.gap_evt.conn_handle,

samples/bluetooth/ble_pwr_profiling/src/main.c

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -329,11 +329,6 @@ static void on_ble_evt(const ble_evt_t *evt, void *ctx)
329329
#endif
330330
break;
331331

332-
case BLE_GAP_EVT_AUTH_STATUS:
333-
LOG_INF("Authentication status: %#x",
334-
evt->evt.gap_evt.params.auth_status.auth_status);
335-
break;
336-
337332
case BLE_GAP_EVT_SEC_PARAMS_REQUEST:
338333
/* Pairing not supported */
339334
nrf_err = sd_ble_gap_sec_params_reply(evt->evt.gap_evt.conn_handle,

samples/boot/mcuboot_recovery_entry/src/main.c

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -72,13 +72,6 @@ static void on_ble_evt(const ble_evt_t *evt, void *ctx)
7272
break;
7373
}
7474

75-
case BLE_GAP_EVT_AUTH_STATUS:
76-
{
77-
LOG_INF("Authentication status: %#x",
78-
evt->evt.gap_evt.params.auth_status.auth_status);
79-
break;
80-
}
81-
8275
case BLE_GAP_EVT_SEC_PARAMS_REQUEST:
8376
{
8477
/* Pairing not supported */

0 commit comments

Comments
 (0)