dresden-elektronik / deconz-rest-plugin

deCONZ REST-API plugin to control ZigBee devices
BSD 3-Clause "New" or "Revised" License
1.89k stars 496 forks source link

CLIPSensors must be visible to the user? [improvement] #7647

Closed mattreim closed 6 months ago

mattreim commented 6 months ago

Does the issue really belong here?

Is there already an existing issue for this?

Describe the bug

These CLIPSensors must be visible to the user? They are actually only intended for internal processing and not for external processing.

Or can you get your own folder? (e.g. FakeSensors;)

Steps to reproduce the behavior

Sensor list

Expected behavior

No FakeSensors to see

Screenshots

RESTED:

CLIPPresence

CLIPGenericStatus

These have been added with version 2.26.1-beta.

CLIPDaylightOffset

ioBroker:

ioBroker

Environment

deCONZ Logs

No response

Additional context

No response

ebaauw commented 6 months ago

The CLIP sensors should definitely be exposed over the API. Whether they should be visible to users is up to the API client interacting with the user, ioBroker in your case?

mattreim commented 6 months ago

Yes, the data is processed with the ioBroker. But the CLIP sensors are stored under sensors and therefore all external programs have this problem.

ebaauw commented 6 months ago

My external programs don't have this problem, as they explicitly offer the feature to interact with CLIP sensors. It's up to the user to configure the program which sensors should be visible. Phoscon as external program doesn't have this issue, as it explicitly filters out CLIP sensors. Exposing CLIP sensors this way is according to spec, and not a bug.

mattreim commented 6 months ago

Ok thanks, then I'll probably have to live with it.

mattreim commented 6 months ago

Just to understand: the API cannot access other folders such as secrets?

DB-Folder

ebaauw commented 6 months ago

This looks like the database. It contains tables, not folders. And its structure is undocumented, and not exposed by the API. See https://dresden-elektronik.github.io/deconz-rest-doc/ for an overview of the API.

Mimiix commented 6 months ago

Closing as this is not a bug