Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Berücksichtigung der Benutzerrechte #2288

Closed
IobUser opened this issue Mar 18, 2023 · 13 comments
Closed

Berücksichtigung der Benutzerrechte #2288

IobUser opened this issue Mar 18, 2023 · 13 comments
Assignees
Labels
bug 🐛 Something isn't working highlight ✨ Highlight of the current release #prio:HIGH ❗ Priority: HIGH :exclamation: v3.2 Roadmap for v3.2

Comments

@IobUser
Copy link

IobUser commented Mar 18, 2023

Ich verwende das PRO-Feature 'Sicherheitseinstellungen>Zugriff Seiten: ioBroker User Management'
Der Zugriff auf die States erfolgt aber trotzdem immer als Benutzer Admin.
Für mich (Haus mit drei Familien) ist die Visualisierung nur brauchbar, wenn die gesetzten Rechte auf die States berücksichtigt werden.
Auch die Änderbarkeit von States sollte davon abhängig sein, ob der Benutzer das Recht dafür hat (nicht dass erst beim Ändern ein Fehler auftritt)
Toll wäre es in dem Zusammenhang, wenn die Sichtbarkeit von Objekten im Jarvis auch abhängig von Benutzerrechten gesetzt werden kann.

@IobUser IobUser added the feature 🌟 Feature request or enhancement label Mar 18, 2023
@assign-milestone-to-issue assign-milestone-to-issue bot added this to the VOTING FEATURE REQUESTS milestone Mar 18, 2023
Repository owner deleted a comment from assign-milestone-to-issue bot Mar 18, 2023
@Zefau Zefau added bug 🐛 Something isn't working and removed feature 🌟 Feature request or enhancement labels Mar 18, 2023
@Zefau Zefau moved this to ROADMAP in jarvis ROADMAP Mar 18, 2023
@Zefau Zefau removed this from the VOTING FEATURE REQUESTS milestone Mar 18, 2023
@Zefau
Copy link
Owner

Zefau commented Feb 11, 2024

@IobUser seit v3.1.x wird der eingeloggte Benutzer genutzt. Passt das so?

@Zefau Zefau added the #status: waiting-for-feedback ⌛ Status: Issue waiting for user feedback label Feb 11, 2024
@IobUser
Copy link
Author

IobUser commented Feb 12, 2024

Ich verwende derzeit die Version 3.1.2 - und da werden die States noch vom admin und nicht vom angemeldeten Benutzer gesetzt.
Bitte um Info, ab welcher Version das geändert ist.
Ich werde dann auf diese Version aktualisieren und Feedback geben.

@Zefau Zefau added #prio:HIGH ❗ Priority: HIGH :exclamation: v3.2 Roadmap for v3.2 and removed #status: waiting-for-feedback ⌛ Status: Issue waiting for user feedback labels Feb 12, 2024
@github-project-automation github-project-automation bot moved this to ROADMAP in jarvis v3.2 Feb 12, 2024
@Zefau Zefau added the #status: implemented-locally 🛠️ Status: Issue has been implemented locally (but not pushed to repository yet) label Jun 20, 2024
@Zefau Zefau removed this from jarvis ROADMAP Jun 20, 2024
@Zefau
Copy link
Owner

Zefau commented Jun 20, 2024

Die Version v3.2.0-beta.16 sollte den Fehler beheben - bitte einmal prüfen. Sofern es behoben ist, gerne das Issue schließen.


The version v3.2.0-beta.16 should fix the bug - please verify. If the bug has been solved, you may close the issue.

@Zefau Zefau added #status: ready-to-test ✔️ Status: Issue fixed and ready-to-test and removed #status: implemented-locally 🛠️ Status: Issue has been implemented locally (but not pushed to repository yet) labels Jun 20, 2024
@Zefau
Copy link
Owner

Zefau commented Jun 20, 2024

Neue Adapter-Konfiguration:

Image

@Zefau
Copy link
Owner

Zefau commented Jun 20, 2024

@mcuiobroker ich mein das Issue hier (auch bzgl. #2669 (comment)).

Fix kommt.

@mcuiobroker
Copy link
Collaborator

Jetzt kommt
image

@mcuiobroker
Copy link
Collaborator

mcuiobroker commented Jun 20, 2024

Jetzt bin ich drin, aber
image

  • jsonTable wir d auch nicht mehr angezeigt
  • alle DPs werden nach TABwechsel als nicht vorhanden angezeigt

image

  • mediaControl auch betroffen wird nicht angezeigt

@Zefau Zefau added #status: implemented-locally 🛠️ Status: Issue has been implemented locally (but not pushed to repository yet) and removed #status: ready-to-test ✔️ Status: Issue fixed and ready-to-test labels Jun 20, 2024
@Zefau
Copy link
Owner

Zefau commented Jun 20, 2024

Die Version v3.2.0-beta.19 sollte den Fehler beheben - bitte einmal prüfen. Sofern es behoben ist, gerne das Issue schließen.


The version v3.2.0-beta.19 should fix the bug - please verify. If the bug has been solved, you may close the issue.

@Zefau Zefau added #status: ready-to-test ✔️ Status: Issue fixed and ready-to-test and removed #status: implemented-locally 🛠️ Status: Issue has been implemented locally (but not pushed to repository yet) labels Jun 20, 2024
@mcuiobroker
Copy link
Collaborator

Sieht erstmal gut aus. Wie kann man jetzt mit Benutzerrechten ein Gerät oder Widget einschränken?

@Zefau
Copy link
Owner

Zefau commented Jun 20, 2024

Die Benutzerrechte greifen wie bekannt auf der Seite oder der Konfiguration. Der Unterschied ist, dass es nun nicht mehr in jarvis direkt, sondern in ioBroker gepflegt wird.

Dadurch fällt die komplette Abhängigkeit zum Web-Adapter weg, über den vorher die Authentifizierung lief. Dadurch sollte nun Reverse-Proxy auf dem jarvis Port uneingeschränkt funktionieren. Außerdem ist dieses Issue hier hoffentlich gelöst - also, dass die Benutzerrechte korrekt beim setState greifen, wenn mit unterschiedlichen Benutzer-Rechten gearbeitet wird.

@mcuiobroker
Copy link
Collaborator

Man kann trotz Benutzer Einschränkung den Switch schalten im Frontend, im ioBroker wird nicht geschaltet.

@Zefau Zefau added #status: implemented-locally 🛠️ Status: Issue has been implemented locally (but not pushed to repository yet) and removed #status: ready-to-test ✔️ Status: Issue fixed and ready-to-test labels Aug 3, 2024
@Zefau
Copy link
Owner

Zefau commented Aug 10, 2024

Die Version v3.2.0-beta.55 sollte den Fehler beheben - bitte einmal prüfen. Sofern es behoben ist, gerne das Issue schließen.


The version v3.2.0-beta.55 should fix the bug - please verify. If the bug has been solved, you may close the issue.

@Zefau Zefau added #status: ready-to-test ✔️ Status: Issue fixed and ready-to-test and removed #status: implemented-locally 🛠️ Status: Issue has been implemented locally (but not pushed to repository yet) labels Aug 10, 2024
@Zefau Zefau closed this as completed Sep 12, 2024
@github-project-automation github-project-automation bot moved this from ROADMAP to DONE in jarvis v3.2 Sep 12, 2024
@Zefau Zefau added highlight ✨ Highlight of the current release and removed #status: ready-to-test ✔️ Status: Issue fixed and ready-to-test labels Sep 12, 2024
@IobUser
Copy link
Author

IobUser commented Jan 8, 2025

Test mit der Version 3.2.0-rc.12:

  • die States werden mit dem angemeldeten Benutzer verändert und nicht mehr als Admin > OK
  • aber auch ohne Recht zum Ändern des States kann geschaltet werden und es wird dann ein Fehler angezeigt
    image
    Hier sollte das Control deaktiviert angezeigt werden

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something isn't working highlight ✨ Highlight of the current release #prio:HIGH ❗ Priority: HIGH :exclamation: v3.2 Roadmap for v3.2
Projects
Status: DONE
Development

No branches or pull requests

3 participants