FIX: Remove measures with attributes name 'id' or 'type' (hotfix 3.4.3)

[AlvaroVega]

Issue https://github.com/telefonicaid/iotagent-node-lib/issues/1485

this should be tested

• [x] with https://github.com/telefonicaid/iotagent-node-lib/pull/1482/
• [x] curl -i -X POST 'http://localhost:7897/iot/json?i=disp1&k=idmaootoin375kl9iifz7472z' -d '{"m": 3, "id": "mierda", "type":"otro"}' -H 'content-type: application/json'

[fgalan]

We should include at same point of the documentation (any suggestion @mapedraza ?) something like this:

Note that id and type are not valid attribute names at Context Broker. Thus, although a id or type will not break the IOT Agent, they are silently ignored and never progress toward Context Broker entities.

or something with a similar meaning.

[mapedraza]

We should include at same point of the documentation (any suggestion @mapedraza ?) something like this:

Note that id and type are not valid attribute names at Context Broker. Thus, although a id or type will not break the IOT Agent, they are silently ignored and never progress toward Context Broker entities.

or something with a similar meaning.

Probably, instead of deleting, renaming them to _id or _type could be useful in the future. The note you propose sounds good

[fgalan]

Probably, instead of deleting, renaming them to _id or _type could be useful in the future

It depends on how strict we want to be with "illegal" measures. Deleting them will strongly disallow its use, while moving them to attributes _id and _type will somehow allow its usage.

In my opinion it's better to delete, but it's not a strong opinion.

[mapedraza]

LGTM