Combining long reads about nimble-arduino HOT 5 CLOSED

Hi, thanks for testing it out 😃 .

Long reads and writes will be implemented shortly, I have the test code in place currently in the bugfix branch, although commented out for now. It needs more work as NimBLE doesn't really provide a great way to detect when a long write or write is in progress so I need to stumble through the source code a bit more. With luck it won't be too much longer.

from nimble-arduino.

I dont know NimBLE, but here is how long read is solved in old library (its in bluetooth specs):
https://github.com/nkolban/esp32-snippets/blob/master/cpp_utils/BLECharacteristic.cpp#L373

IIRC long write is handled by WRITE_EVT and param->write.is_prep, and finally by EXEC_WRITE_EVT (im not sure about last event).

PS i am wondering if its required to write long read/write code or its maybe handled internally by NimBLE

from nimble-arduino.

@chegewara Thanks for the link, sadly it doesn't seem NimBLE provides any sort of indication flag for long read/write.

The best I can do it seems is compare the data size with the mtu the check the packet header length that may give an indication if this is an initial read or followup. I think it does handle the offset internally though so hopefully that won't need to be managed.

from nimble-arduino.

I figured out I could fix my case by negotiating a higher mtu from the client side.

I'm not sure if that should be the standard way to do it or if the package should somehow combine it.

from nimble-arduino.

Long reads and writes are sub-optimal, they consume a lot of time. It's much better to use the biggest MTU size you can handle.

That said, I have sorted out the long read/write situation in the server code and just now pushed the update to the bugfix branch if you want to check it out.

from nimble-arduino.