dhalucario/badgemagic-rs
1
0
Fork 0
mirror of https://github.com/fossasia/badgemagic-rs synced 2025-06-24 07:43:58 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Remove redundant BLE service UUID check as it is already done using a ScanFilter

Browse source
This commit is contained in:
Lilian 2024-08-01 22:18:11 +02:00
parent b025624958
commit adb2018df9
No known key found for this signature in database
GPG key ID: 1650987A001CB2CB
1 changed files with 6 additions and 13 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

19
src/ble.rs
View file

@ -71,22 +71,15 @@ impl Device {
}
async fn from_peripheral(peripheral: Peripheral) -> Option<Self> {
// Check whether the BLE device has the service UUID we're looking for
// and also the correct name.
// The service uuid is also by devices that are not LED badges, so
// the name check is also necessary.
// The existance of the service with the correct UUID
// exists is already checked by the scan filter.
// But we also need to check the device name to make sure
// we're talking to a badge as some devices that are not led badges
// also use the same service UUID.
let props = peripheral.properties().await.ok()??;
let local_name = props.local_name.as_ref()?;
if local_name != BADGE_BLE_DEVICE_NAME {
return None;
}
if props
.services
.iter()
.any(|uuid| *uuid == BADGE_SERVICE_UUID)
{
if local_name == BADGE_BLE_DEVICE_NAME {
Some(Self { peripheral })
} else {
None