Change the argument of set_auth function to bitflag

CHANGELOG.md

@@ -2,6 +2,7 @@
 - Upgraded to `esp-idf-hal` 0.42 and `esp-idf-svc` 0.47
 - Added Self parameter to BLEScan::on_result.
 - Add feature for building with std support ([#36](https://github.com/taks/esp32-nimble/pull/36))
+- Change the argument of set_auth function to bitflag.
 
 ## [0.2.2]
 - Fix advertising regression in v0.2.1 ([#31](https://github.com/taks/esp32-nimble/pull/31))

examples/ble_keyboard.rs

@@ -227,7 +227,7 @@ impl Keyboard {
     let device = BLEDevice::take();
     device
       .security()
-      .set_auth(true, true, true)
+      .set_auth(AuthReq::all())
       .set_io_cap(SecurityIOCap::NoInputNoOutput);
 
     let server = device.get_server();

examples/ble_secure_client.rs

@@ -21,7 +21,7 @@ fn main() {
       .unwrap();
     device
       .security()
-      .set_auth(true, true, true)
+      .set_auth(AuthReq::all())
       .set_io_cap(SecurityIOCap::KeyboardOnly);
 
     let ble_scan = device.get_scan();

examples/ble_secure_server.rs

@@ -8,7 +8,7 @@ fn main() {
   let device = BLEDevice::take();
   device
     .security()
-    .set_auth(true, true, true)
+    .set_auth(AuthReq::all())
     .set_passkey(123456)
     .set_io_cap(SecurityIOCap::DisplayOnly);
 

src/ble_security.rs

@@ -8,11 +8,11 @@ impl BLESecurity {
     Self { passkey: 0 }
   }
 
-  pub fn set_auth(&mut self, bonding: bool, mitm: bool, sc: bool) -> &mut Self {
+  pub fn set_auth(&mut self, auth_req: enums::AuthReq) -> &mut Self {
     unsafe {
-      esp_idf_sys::ble_hs_cfg.set_sm_bonding(bonding as _);
-      esp_idf_sys::ble_hs_cfg.set_sm_mitm(mitm as _);
-      esp_idf_sys::ble_hs_cfg.set_sm_sc(sc as _);
+      esp_idf_sys::ble_hs_cfg.set_sm_bonding(auth_req.contains(enums::AuthReq::Bond) as _);
+      esp_idf_sys::ble_hs_cfg.set_sm_mitm(auth_req.contains(enums::AuthReq::Mitm) as _);
+      esp_idf_sys::ble_hs_cfg.set_sm_sc(auth_req.contains(enums::AuthReq::Sc) as _);
     }
 
     self

src/enums.rs

@@ -1,3 +1,4 @@
+use bitflags::bitflags;
 use esp_idf_sys::*;
 
 #[repr(u8)]
@@ -79,3 +80,15 @@ pub enum PowerType {
   /// For default, if not set other, it will use default value
   Default = esp_ble_power_type_t_ESP_BLE_PWR_TYPE_DEFAULT as _,
 }
+
+bitflags! {
+  #[derive(Debug, Clone, Copy, PartialEq, Eq)]
+  pub struct AuthReq: u8 {
+    /// allow bounding
+    const Bond = 0b001;
+    /// man in the middle protection
+    const Mitm = 0b010;
+    /// secure connection pairing
+    const Sc = 0b100;
+  }
+}