SierraWireless / octave-orp

Octave Resource Protocol
MIT License
1 stars 4 forks source link

Fix notifications display #14

Closed fdur closed 1 year ago

fdur commented 1 year ago

Fix notification display. Such notifications do not use the byte1. Add a check if this byte is used or not.

fdur commented 1 year ago

Using python client:

> add handler /app/cloudInterface/connected/value
Sending: H.01P/app/cloudInterface/connected/value
Received     : h@
Message type : response add handler
Status       : OK
Sequence     : 1

> Received     : cT1668087250.005864,P/app/cloudInterface/connected/value,Dfalse
Message type : handler call
Status       : NOT IMPLEMENTED
Sequence     : 1
Timestamp    : 1668087250.005864
Path         : /app/cloudInterface/connected/value
Data         : false

Acknowledged push notification

>
Received     : cT1668087264.986161,P/app/cloudInterface/connected/value,Dtrue
Message type : handler call
Status       : NOT IMPLEMENTED
Sequence     : 2
Timestamp    : 1668087264.986161
Path         : /app/cloudInterface/connected/value
Data         : true

Acknowledged push notification
>
Nico26-sw commented 1 year ago

Python client looks OK (except the "NOT IMPLEMENTED" print but this is minor) Thanks