drogue-cloud icon indicating copy to clipboard operation
drogue-cloud copied to clipboard

TTN fields are not validated

Open lulf opened this issue 2 years ago • 4 comments

Creating a device in TTN via the TTN integration in drogue cloud works, and I'm able to connect the device. But I'm noticing an error in the status of the device object:

"reason": "Request failed: 400 Bad Request: {\"code\":3,\"message\":\"error:pkg/rpcmiddleware/validator:field_mask_paths (forbidden path(s) in field mask)\",\"details\":[{\"@type\":\"type.googleapis.com/ttn.lorawan.v3.ErrorDetails\",\"namespace\":\"pkg/rpcmiddleware/validator\",\"name\":\"field_mask_paths\",\"message_format\":\"forbidden path(s) in field mask\",\"attributes\":{\"forbidden_paths\":[\"ids.dev_eui\",\"ids.join_eui\"]},\"correlation_id\":\"6c085ca7174647f2a797749437990374\",\"code\":3}]}",

The EUI fields of the ttn spec should be validated to prevent them being changed.

lulf avatar Jun 22 '22 11:06 lulf

The underlying cause: https://github.com/TheThingsNetwork/lorawan-stack/issues/5553

We need to update the reconciling loop to not reconcile these fields.

lulf avatar Jun 29 '22 08:06 lulf

I would re-open it, was we didn't properly fix it.

ctron avatar Jul 01 '22 13:07 ctron

Ok, I mentioned creating a new issue for the webhooks in #301 but I can edit the description of this one instead.

lulf avatar Jul 01 '22 13:07 lulf

Ah ok. Works both.

ctron avatar Jul 01 '22 14:07 ctron