provisioning: Refactor transaction (un)packing

SilvairGit / python-bluetooth-mesh

https://pypi.org/project/bluetooth-mesh/

GNU General Public License v2.0

71 stars 22 forks source link

provisioning: Refactor transaction (un)packing #153

Closed michallowasrzechonek-silvair closed 2 years ago

michallowasrzechonek-silvair commented 2 years ago

@tomaszlelental-silvair może tyle wystarczy?

Z tego co widzę w specu, to nie można mieć otwartej >1 transakcji jednocześnie, więc algorytm wyglądałby tak:

słuchaj rzeczy z radia i zbieraj wiadomości GenericProvisioning do listy
wrzucaj listę do ProvisioningTransaction.unpack - jeśli rzuci wyjątkiem, kontynuuj
jeśli nie rzuci, wyślij TransactionAck i obsłuż ProvisioningPDU

Czy potrzebujesz też obsługę link i transaction id?

michallowasrzechonek-silvair commented 2 years ago

@tomaszlelental-silvair rzuć okiem czy takie API by Ci pasowało - jeszcze muszę testy dopisać

tomaszlelental-silvair commented 2 years ago

Dzięki za nową wersję. To idzie w stronę całkiem sensownego provisionera "produkcyjnego", a w testach potrzebuję przede wszystkim psuć :) , czyli np. wysłać inne Transaction Number niż wynikałoby ze Speca albo nie wysłać ACKa w ogóle. Co sądzisz o wprowadzeniu do metod opcjonalnych parametrów, które, jeśli podane w wywołaniu, nadpisywałyby wartości obliczone wewnętrznie? Ewentualnie przeciążę/rozszerzę to w moim frameworku testowym.

michallowasrzechonek-silvair commented 2 years ago

To zrobię żebyś acki musiał wysłać sam i dodam parametry.

Segmenty możesz uszkodzić też po wygenerowaniu, przed wypchnięciem na radio.