Support ParameterReq with a populated requests
field but an empty param
and value
field
#20
Labels
enhancement
New feature or request
fjage defines that a
ParameterReq
should always have theparam
field set, and only use therequests
field for the 2nd parameter onwards. However, the Fjage.java implementation does support aParameterReq
with all the parameters set in therequests
field and an emptyparam
field.We should update
Fjage.jl
to follow Fjage.java implementation and accept such messages as well.For example..
The correct way..
1657162337106|FINE|org.arl.fjage.remote.ConnectionHandler@115:run|tcp:///10.30.0.229:1100//10.30.0.248.59508 <<< {"id":"49916647-d700-49b1-9c5e-e46d2eed19eb","action":"send","message":{"clazz":"org.arl.fjage.param.ParameterReq","data":{"index":-1,"requests":null,"param":"pbscnt","value":1,"msgID":"49916647-d700-49b1-9c5e-e46d2eed19eb","perf":"REQUEST","recipient":"bb","sender":"unetsh#-ly2bin","inReplyTo":null,"sentAt":1657162337054}},"relay":true}
vs the incorrect way (which I propose should be supported)
1657161986677|FINE|org.arl.fjage.remote.ConnectionHandler@100:run|tcp:///10.30.0.229:1100//10.30.0.248.59346 <<< {"action": "send", "relay": true, "message": { "clazz": "org.arl.fjage.param.ParameterReq", "data": {"msgID":"3e0a4748-bfd7-4f95-b3bd-04112524707a","perf":"REQUEST","recipient":"bb","sender":"PythonGW-422a2914-cd67-4b66-a045-b47c03d67716","inReplyTo":null,"index":-1,"requests":[{"param":"pbscnt","value":1}]} }}
The text was updated successfully, but these errors were encountered: