RaspberryMatic - Verbesserungsvorschläge/Wünsche

[jp112sdl]

Also um es kurz zu machen:
Wenn man in /www/rega/pages/tabs/admin/userAdministration.htm:L147 die Bedingung

Code: Alles auswählen

&& (user.Name()!='Admin')

entfernt, kann man auch über die WebUI den "Admin" löschen.

Den eigenen Ast absägen (bzw. den letzten verbleibenden Admin löschen) kann man nicht, da man
a) nur mit Administratorenberechtigung löschen und
b) nur andere Nutzer, jedoch nicht den eigenen löschen darf
Siehe

Code: Alles auswählen

if ((system.GetSessionVar('sessionUserID') != user.ID())...

in der selben Zeile.

Eine fixe Verdrahtung zu dem Benutzer "Admin" oder der User ID "1004" finde ich nirgends im WebUI-Code. Außer an der Stelle, wo die Admin-Edit-Felder deaktiviert werden, aber das ist nur Kosmetik

Wäre einzig noch die ReGa, ob die irgendwo Wert drauf legt, dass es den Benutzer und/oder die UserID gibt.

[jp112sdl]

es wäre schön, wenn man den User "Admin" löschen könnte, sobald man einen anderen User mit Admin-Rechte generiert hat.

Dann verwandle das in ein entsprechendes Enhancement Ticket im RaspberryMatic GitHub und dann schauen wir mal wohin die Reise geht.

Ich würde dann einen PR mit meiner vorgestellten Änderung machen ^^

[jmaus]

es wäre schön, wenn man den User "Admin" löschen könnte, sobald man einen anderen User mit Admin-Rechte generiert hat.

Dann verwandle das in ein entsprechendes Enhancement Ticket im RaspberryMatic GitHub und dann schauen wir mal wohin die Reise geht.

Ich würde dann einen PR mit meiner vorgestellten Änderung machen ^^

Immer wieder gerne Jerome! Sind da ja bereits ein recht gut eingespieltes Team wie es aussieht...

... und wenn du dabei bist, vielleicht machst du doch gleich auch einen PR für deine vorgeschlagenen Fix für den momentan nicht funktionierenden Gerätetausch? Von eQ3 hab ich diesbzgl. keine Rückmeldung, dein Vorschlag eines Fixes würde ich jedoch trotzdem gerne einmal breiter testen lassen.

[jp112sdl]

... und wenn du dabei bist, vielleicht machst du doch gleich auch einen PR für deine vorgeschlagenen Fix für den momentan nicht funktionierenden Gerätetausch?

Ja, mach ich. Jetzt erst recht

[jp112sdl]

Ich weiß nicht, ob das schon behandelt wurde. ich möchte es aber trotzdem noch anmerken:

es wäre schön, wenn man den User "Admin" löschen könnte, sobald man einen anderen User mit Admin-Rechte generiert hat.
jede CCU, RM, piVCCU oder debmatic hat als User den Admin drin. Der ist für Eindringlinge sehr schwer zu erraten.
Sollte der aber fehlen, wird es schon schwieriger.

Dann verwandle das in ein entsprechendes Enhancement Ticket im RaspberryMatic GitHub und dann schauen wir mal wohin die Reise geht.

@MathiasZ : Würdest du das Ticket im Github noch erstellen, damit ich den PullRequest darauf referenzieren kann?

[MathiasZ]

@jp112sdl
Mache ich sobald ich mit dem PC wieder online bin.
Am PC ist Github kein Problem.
Am Handy ist Github eher .
Sorry.....

[MathiasZ]

Also um es kurz zu machen:
Wenn man in /www/rega/pages/tabs/admin/userAdministration.htm:L147 die Bedingung

Code: Alles auswählen

&& (user.Name()!='Admin')

entfernt, kann man auch über die WebUI den "Admin" löschen.

Den eigenen Ast absägen (bzw. den letzten verbleibenden Admin löschen) kann man nicht, da man
a) nur mit Administratorenberechtigung löschen und
b) nur andere Nutzer, jedoch nicht den eigenen löschen darf
Siehe

Code: Alles auswählen

if ((system.GetSessionVar('sessionUserID') != user.ID())...

in der selben Zeile.

Eine fixe Verdrahtung zu dem Benutzer "Admin" oder der User ID "1004" finde ich nirgends im WebUI-Code. Außer an der Stelle, wo die Admin-Edit-Felder deaktiviert werden, aber das ist nur Kosmetik

Wäre einzig noch die ReGa, ob die irgendwo Wert drauf legt, dass es den Benutzer und/oder die UserID gibt.

Ich war einmal am spielen
Erst wollte ich den Benutzername 'Admin' in 'benuzerdefiniert' ändern,

Code: Alles auswählen

&& (user.Name()!='benutzerdefiniert')

dann den Eintrag:

Code: Alles auswählen

&& (user.Name()!='Admin')

löschen.
Gibt es da einen speziellen Schreibschutz?
Ich konnte das weder mit WINSCP noch mit Putty ändern.
Gruß,
Mathias

[jp112sdl]

Gibt es da einen speziellen Schreibschutz?

Ja, die root-Partition ist üblicherweise nur lesend gemountet.

Mach mal das Ticket, dann mach ich den PR, dann gibt es einen nightly-build und du kannst testen

EDIT: Grad gesehen

[MathiasZ]

schon passiert....

[jp112sdl]

Im Zuge der Patch-Aufbereitung sind mir 2 Dinge aufgefallen, die ich gern erstmal gern von jemandem gegengeprüft hätte.

1.) Beim Anlegen eines neuen Benutzers (beim Klick auf den Button "Neu") erfolgt ein Eintrag in /var/log/messages mit

Code: Alles auswählen

ReGaHss: ERROR: ScriptRuntimeError: var userID = "0"; var _session_id_ = "sRM6qhuLYm";    var user = dom.GetObject(userID);   Write(user.UserFirstName());  [ExecError():iseESPexec.cpp:12879]

2.) Beim Löschen eines Benutzers erfolgt ein Eintrag in /var/log/messages mit

Code: Alles auswählen

ReGaHss: ERROR: invalid parameter; id = 65535 [ExecEnum():iseESPexec.cpp:6348]

Hab ich mir hier jetzt was kaputt gespielt oder ist das auch in einem nicht-verfummelten System so?