Mqtt observable property is not exposed

[hasanheroglu]

https://github.com/eclipse/thingweb.node-wot/blob/09dfd6ab03805b7b39ab39433880496466d25e67/packages/binding-mqtt/src/mqtt-broker-server.ts#L123-L149

In case property has observable: true and writeOnly: true fields form for observable not exposed. Additionally, if readOnly: true at the same time, no forms would be exposed.

It could be better to have the code like this:

if (property.observable) {
    // observable form expose logic
}
if (property.writeOnly) {
    // writeOnly form expose logic
}

And I think, we can skip totally readOnly since readproperty is not used with mqtt.

[hasanheroglu]

@egekorkan Should we remove the read logic here?

[egekorkan]

@hasanheroglu there should be no form with readproperty exposed. There is a spec-level issue for reading as explained at #1241 (comment)

[relu91]

Wait can't we support read operations only if we have the retain flag to true? not wonderful but for some brownfield devices might work.

[egekorkan]

We can deliver the value wished to be read if it is retained for X amount of time or was published within a timeout Y (subscription is sent, Y has passed, read fails). Maybe those can be parameters to be passed in the options where those parameters are specific to node-wot. There can be of course default values for both.