@BartTK : You describe passive scans with your PR (i.e. improvement in crowded BLE environments).

If I want to use the passive scan mode in crowded BLE environments, can you please tell me the best way ?

1. Do I call BLE.scanForUuid(MY_BLE_UUID_SERVICE, true, false); ??

(i.e. withDuplicates = true and activeScan = false)

or do I call BLE.scanForUuid(MY_BLE_UUID_SERVICE, false, false); ??

(i.e. withDuplicates = false and activeScan = false)

2. Can you please confirm that in any case, if I use your PR-chage with activeScan = false, then I have to call BLE.available(true) in any case ?

3. Do you use BLE.poll(); right in front of BLE.available(true); ?? (some people seem to recommend this). Is it necessary with your PR-change anymore ?

4. I realized that using your activeScan = false, then I cannot use peripheral.localName() anymore to check whether my sender-peripheral has the correct name. Can you confirm that with your PR-change there is no longer the possibility to use peripheral.localName() ?

5. Are you aware of the fact that if you use your PR-change that the problematic of freezing receive-centrals still exists ?
   See this issue for more information - it is still not solved.
   What I mean by freezing receive-central is that with the ArduinoBLE, there is a problem when the receiver-central is on 24/7. And the sender-pheripheral is turned off (cut from battery) and turned on again after hours. This is an issue that is still not solved. Also not with your PR-change.

Thank you for your clarifications on this.

I use BLE.scanForAddress(DEVICE_ADDRESS, true, false); myself. Followed by a for-loop that keeps calling BLE.available(true); until it returns a device; that must be the correct device as I was scanning very specifically for the address. There is no BLE.poll(); in there, the BLE.available(true); takes care of that itself.

When doing a passive scan, you get less information from a device compared to an active scan. I think it is device dependent what it returns exactly. Local names are typically not included, so you shouldn't assume that those are present. It has been a while since I played around with it, but I think the service UUID is included. My guess would be that your BLE.scanForUuid(..); should work.

I've only added support for passive scans and haven't done anything about that issue you mentioned, so it wasn't my intent to solve that problem here.

@facchinm All checks succeeded, the review was OK and there are no conflicts. Is there anything I still need to do to have this PR merged?

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

---

BartTK seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}