Schüler können keine custom-Felder in der WebUI bearbeiten

jochen · 6. August 2022 um 18:57

bitte erst NACH Deinem Urlaub beantworten!

Ich habe festgestellt, dass bei uns zwar Lehrer die Einstellungen für Custom-Felder ändern können, nicht aber Schüler, obwohl das in den Einstellungen für Schüler aktiviert wurde. Sie sehen das Custom-Feld auf der Hauptseite, können es aber nicht bearbeiten.
Ist das ein Bug oder mache ich etwas falsch?

Vielen Dank und viele Grüße,
Jochen

Arnaud · 6. August 2022 um 22:02

Hallo Jochen,

Ich fahre bald in Urlaub, bin aber noch da

Es stimmt was du sagst, ich habe vergessen die Abstimmung bei den anderen @Entwickler letzte Woche zu fragen. Alles ist schon vorbereitet, nur der Zugang ist zu den Schüler momentan gesperrt :

github.com

linuxmuster/linuxmuster-webui7/blob/efc63fcd1183e500eb2b0800d6634ff18716a2de/usr/lib/linuxmuster-webui/plugins/lmn_landingpage/resources/partial/index.html#L96

    
      
                                      <span style="position:absolute;right:10px;" ng:show="!isListAttr(custom.attr)">
                                          {{custom.value}}
                                      </span>
                                      <div ng:show="isListAttr(custom.attr)" style="position:absolute;right:10px;">
                                          <span ng:repeat="value in custom.value track by $index">
                                              {{value}}
                                          </span>
                                      </div>
                                  </li>
                              </ul>
                              <button style="margin-top:20px;" class="btn btn-primary pull-right" ng:click="changeCustomFields()" ng:show="user.sophomorixRole != 'student'">
                                  <span translate>Change my data</span>
                              </button>
                          </div>
                      </div>
                  </div>
              </div>
          
          

          

          
</div>

Ich wollte allein den Zugang zur Kommando sophomorix-user an die Schüler erlauben, wir müssen es noch kurz absprechen.

Gruß

Arnaud

jochen · 8. August 2022 um 06:41

Danke Dir, kein Stress und einen schönen und erholsamen Urlaub!!!

Viele Grüße,
Jochen

jochen · 27. August 2022 um 12:44

Hallo Arnaud,

könnte(s)t Du/Ihr das (nach! Deinem Urlaub) bis zum Schulstart bitte noch aktivieren, falls das keine größeren Umbauarbeiten erfordern sollte? Ich würde das gerne in meinen Leitfaden für neue Klassen übernehmen.

Vielen Dank und viele Grüße,
Jochen

Arnaud · 29. August 2022 um 10:42

Hallo,

@jeffbeck und @Till : sieht ihr etwas dagegen ?

Gruß

Arnaud

Till · 30. August 2022 um 14:05

Wirklich sauber wäre dass nicht wenn ich es richtig sehe. Mit dem Befehl können Schüler schon einige Optionen ändern wenn Sie eine entsprechende Shell bekommen, oder sehe ich das falsch?

dorian · 30. August 2022 um 17:46

Wäre es nicht sogar so, dass sie dann Zugriff auf einen ldap Admin account bräuchten, damit das sophomorix-user script überhaupt funktioniert?

Till · 30. August 2022 um 17:47

Die Rechte werden über sudo eingeschränkt auf das entsprechende Kommando. Dieses Kommando würde also mit superuser Rechten ausgeführt werden.

dorian · 30. August 2022 um 17:49

Achso, ok. Ist sophomorix denn gegen diverse injections gehärtet? (@jeffbeck ?)

jochen · 30. August 2022 um 18:26

Hallo zusammen,

die technischen Hintergründe überblicke ich natürlich nicht aber vielleicht zu unserem Hintergrund:
wir haben keine Schul-E-Mail-Adressen, die Standardadresse login@domain.de funktioniert bei uns also nicht. Wir sind darauf angewiesen, alle SuS über deren private E-Mail-Adresse erreichen zu können. Nextcloud liest dazu die custom-E-Mail der KuK bereits aus und verwendet diese, für Moodle und evtl. andere Dienste wäre das auch noch praktisch. SuS können diese leider noch nicht selbst setzen und das für alle SuS als Admin zu machen geht nicht. Daher wäre es genial, wenn die SuS diese Möglichkeit ebenfalls bekommen könnten. Die KuK können das ja auch und bezüglich Sicherheit würde ich da manchmal keinen Unterschied zwischen diesen Gruppen sehen

Vielen Dank und viele Grüße,
Jochen

Arnaud · 30. August 2022 um 20:02

Ja genau, aber das ist schon der Fall seit 2 Jahre, deswegen frage ich noch ein mal :

Ich weiss nicht genau, warum diese Änderung vorgekommen ist, oder ob es ein Fehler ist. Kannst du dich erinnern ?

Eigentlich sind wir in die Webui diejenige, die Variabeln in sophomorix-Kommandos einsetzen, und das sollten wir jedes mal sauber überprüfen. Aber klar, man kann immer einen Fehler machen, das passiert mir oft.

Gruß

Arnaud

Arnaud · 30. August 2022 um 20:06

Yep Jochen, deswegen ist diese Möglichkeit von CustomFelder in sophomorix einfach sehr gut. Wir sollten trotzdem wirklich überprüfen, dass es sauber funktioniert.

Am schlimmtens, wenn du die Adresse in einem CSV gesammelt hast, kann man auch sehr schnell es skripten. Ich sehe dich schon schwitzen : klar, die Adresse sammeln ist nicht das gleiche wie wenn die Schüler es selber eintragen, aber das wäre die günstigere alternative bis wir es lösen

Gruß

Arnaud

jochen · 30. August 2022 um 20:28

Hallo Arnaud,

das wird bei uns praktisch kaum möglich sein. Blockunterricht etc. Dann hoffe ich auf eine Lösung und warte mal ab.

Danke und viele Grüße,
Jochen

dorian · 31. August 2022 um 08:21

Eine Sache noch:
Kann man mit sophomorix-user nicht jeden beliebigen User bearbeiten und auch die eigene Quota verändern?

Im Linuxclient stelle ich sicher, dass user nur dinge verändern können, die sie selbst betreffen, indem ich aus der SUDO_USER env-Variable den tatsächlcihen user rauslese:

github.com

linuxmuster/linuxmuster-linuxclient7/blob/227e4c92836451ed9ecdcfffd1d82faa01631529/usr/share/linuxmuster-linuxclient7/scripts/sudoTools#L36

    
      
          requiredGroupCache.add_argument("--name", help="The name of the share", required=True)
          
          
args = parser.parse_args()
          
          
task = args.task
          
          
if not os.geteuid() == 0:
              print("This script has to be run using sudo!")
              exit(1)
          
          
username = os.getenv("SUDO_USER")
          
          
if task == "install-printer":
              if printers.installPrinter(args.path, name=args.name, username=username):
                  sys.exit(0)
              else:
                  sys.exit(1)
              pass
          elif task == "mount-share":
              if shares._mountShare(username, args.path, args.name, args.hidden, False):
                  sys.exit(0)

jochen · 3. September 2022 um 10:15

Wie sind Dorians Antworten und das neue Paket von Thomas mit dem fix-sudoers zu interpretieren? Dass die Funktionalität doch zeitnah kommen könnte oder ist das was komplett anderes? Ich frag so ein bisschen penetrant, weil ich am Di die Doku für die SuS dazu fertig haben muss.

Danke und liebe Grüße,
Jochen

Arnaud · 3. September 2022 um 13:26

Hallo Jochen,

Die Änderung von custom felder durch SuS wird später kommen, wir sollen uns erst mal einigen, wie wir es am besten und saubersten einbauen.

Gruß

Arnaud

dorian · 3. September 2022 um 13:43

Für die Zwischenzeit hilft dir vielleicht die Landingpage:

Die hab ich genau für diesen usecase entwickelt.

VG,
Dorian

jochen · 3. September 2022 um 18:03

Hallo Arnaud und Dorian,

danke Euch für die Info. Dann müssen wir noch etwas warten (vielleicht auf die 7.2?) oder uns nochmal mit der Landingpage befassen. Das hatten wir schon mal versucht, hat aber nicht auf Anhieb geklappt.

Viele Grüße,
Jochen