GitLab

- Remove client from the list of devices in case registration fails
- Filter other events not related to registration during registration phase
- Cleanup if connect fails
- Reset if after disconnect
- Disconnect callback *after* cleanup is done so object can be deleted

This fixes some of the issues I had like:
- `BLEClient::connect` hangs up and never recovered because registration failed
- `BLEClient` could not be deleted after disconnect or deletion creating ghost events https://github.com/espressif/arduino-esp32/issues/4047
- `BLEClient` could not be properly reused after a connection was attempted (successful or not) 

* Cleanup in case of registration and connect failure.
Cleanup before calling disconnect callback for safe delete.
Reject other events during registration.
Adresses #4047, #4055

* Clear if after unregister #4047

fixes #4596

* Prevent possible undefined behaviour by get methods not taking an index as parameter
* Add methods to get the count of service data UUIDs and service UUIDs
* Various code improvements

The variable is useful when debugging AsyncUDP send problems.
The upper application can read and analyze the error reason.

* Fixed crash on delete after disconnect

* Fixed memory leak when getting characteristics

* Removed guard
Co-authored-by: ushiboy <ushiboy.dev@gmail.com>

* Speed up upload by a factor of 17

Uploads are very slow because of an unnecessary "client.connected()" check in _uploadReadByte().

Here is what happens:
client.connected() is called for every byte read.  WiFiClient::connected() calls recv(fd(), &dummy, 0, MSG_DONTWAIT); which takes a relatively long time, so the optimized path of returning a buffered byte via client.read() is effectively nullified.

Removing the one line changed the upload speed for a 2 MB file (discarding the received data) from 22 KB/sec (before) to 367 KB/sec (after).

The change is safe in the face of disconnects because client.read(), when it no longer has buffered data, calls (WiFiClient)  fillBuffer(), which calls recv(), so the disconnection will be detected in due course.

* Move disconnect check into the timeout loop

* Update IDF to d8082b7f3

Fixes: https://github.com/espressif/arduino-esp32/issues/4763

Fixes: https://github.com/espressif/arduino-esp32/issues/2537

Fixes: https://github.com/espressif/arduino-esp32/issues/4732

Co-authored-by: me-no-dev <me-no-dev@github.com>

Co-authored-by: me-no-dev <me-no-dev@github.com>

esp-face: master 420fc7e
esp32-camera: master 0107093

Added authentication callbacks and example, resolves #4622.

BLERemoteCharacteristic::registerForNotify: Permit event registration without updating descriptor. (#4659)

added availableForWrite to Print.h to make compatible with Arduino Print.h

Compiler complains on unused parameter pCharacteristicMap:

framework-arduinoespressif32\libraries\BLE\src\BLERemoteService.cpp: In member function 'void BLERemoteService::getCharacteristics(std::map<short unsigned int, BLERemoteCharacteristic*>*)':
\framework-arduinoespressif32\libraries\BLE\src\BLERemoteService.cpp:246:89: warning: parameter 'pCharacteristicMap' set but not used [-Wunused-but-set-parameter]
 void BLERemoteService::getCharacteristics(std::map<uint16_t, BLERemoteCharacteristic*>* pCharacteristicMap) {

esp-face: master 420fc7e
esp32-camera: master 0107093

Follow up for https://github.com/espressif/arduino-esp32/pull/4564

Added guarded define to set the stacksize on the main looptask.
Advantage of this is that build_flags can be used to provide a different value for the stack size should it be neccessary
default behaviour is unaffected

Replace (depricated) bzero, bcopy with memset and memcpy.

Replace (depricated) bzero with memset.