Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support to disable/enable white list control #615

Closed
wants to merge 39 commits into from

Conversation

ahome-wugu
Copy link

#331 It would be more convenient to manage white list by adding config message to temporarily disable the already set white list and enable later , e.g. {"white-list":"false"} and {"white-list":"true"}, because sometimes we may need to observe the new approaching BLE mac via mqtt gateway by temp disable the already set white list, and add the new desire one and then enable the white list again, fyi !

1technophile and others added 30 commits January 11, 2020 20:23
#529
add a test environment so as to verify this functionnality on [CI]
* Add support for weather stations
* Add configuration for ESP8266
* Lean on Arduino log library instead of custom logging methods
Alarm clock with BLE temperature and humidity measurement
booleans can not interpreted as null terminated strings
* merge duplicate code into function setWorBMac

* rename setWorBMac to updateWorB

* add helper getDeviceByMac

* change updateWorB to use helper getDeviceByMac

* change isWhite to use helper getDeviceByMac

* change isBlack to use helper getDeviceByMac

* change isDiscovered to use helper getDeviceByMac

* eliminate duplicate devices walk for isWhite, isBlack and isDiscovered check

* fix a data corruption race between MQTTtoBT and discovery-functions

the ***Discovery functions every time create a new device unless this device is set to white-list before. as a result of this, the devices list contains the same mac multiple times with different flags

* eliminate duplicate devices walk for oneWhite check

because now we have a utility function 'createOrUpdateDevice' we can globally cache this as a single flag

* secure createOrUpdateDevice agains data races on multicore esp32

* fix the concept of the createOrUpdateDevice

c / c++ does not allow NULL as function parameter like modern languages

* use nkolban Semaphore helper class (already included and used from eos1d3/ESP32_BLE librarie)

this makes the semaphore handling more cleaner and readable

* merge duplicate code during createDiscovery into new helper createDiscoveryFromList

* replace excessive use of ZmqttDiscovery precompiler directive in ble advertised callbacks with dummy functions

for a better readability we can remove this because isDiscovered does not do a devices walk anymore

* fix build errors found by travis-ci
#528

as it is generating conflict with TRIGGER_PIN (used for reseting and erasing the flash)
Removed device_class values unknown to Home-Assistant. Fixes #570 and fixes #571.
enable after ATTEMPTS_BEFORE_BG
and  ATTEMPTS_BEFORE_BG to try another Wifi protocol
 #563
#563  Move protocol change method outside of wifimanager scope
and correct build error for ESP8266
Add the necessary macros and move the new OTA function to an appropriate location (for ESP only)
* adds CC1101 (433mhz) support esp32 and esp8266
#579
Update doc to reflect necessary manual changes
Implement a part of these changes to automatic script
#564
replace %s -> %d for LORA pin numbers
jmw6773 and others added 9 commits April 29, 2020 08:22
* Added HTU21 Sensor Support

Co-authored-by: Jonathan Wells <[email protected]>
* Add condition to check Low Power Mode

* Added WeatherStation libs platform.ini
Line44: typo (paried > paired)
* Allow pins/settings to be defined in INI file

* Open to make publishing ManufacturerData optional
* Add infos and change HM10 speed to a macro
* remove unnecessaries macros
@ahome-wugu ahome-wugu closed this May 13, 2020
@ahome-wugu
Copy link
Author

sorry, not yet familiarize how to submit PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.