Univention Bugzilla – Bug 45332
Check school admin accounts
Last modified: 2020-10-12 16:19:14 CEST
The diagnostic module should check if school admin accounts are correctly configured: - user is member of OU admin group - user object contains object class ucsschoolAdministrator
Added diagnosis module with commit commit b6a08c6615da6e72f9c5ce11e6d04ac97064f1fb Bug #45332: added diagnosis module for admin check to branch troehmey/bug45332_check_school_admins
QA -> REOPEN with some remarks - In the first loop you only need/ use ucsschoolSchool - If ucsschoolSchool is not set, a key-error is catched. Since the admin is a school admin, I would like to have, maybe slightly modified, message about this. - if not admin["schools"] -> you don't really need this case - I would skip the second for admin in admins loop and add the problematic objects at the end of the "admin" iteration. ... other than that the code seems to be working.
Resolved issues with fixup commit: c839e55c0 Bug #45332: added diagnosis module for admin check
Added one more fixup which checks that each member of an admin-school group is a ucsschoolAdministrator
QA -> All Ok -> REOPEN for merge & build Thanks for the changes! As discussed, squash the commits, merge, add changelogs & yaml & build
Applied squash, merge and build. Commits: ffd692a0f Bug #45332: added yaml a04efd470 Bug #45332: added changelog entry 3f7422387 Bug #45332: Merge branch 'troehmey/bug45332_check_school_admins' into 4.4 Successful build: Package: ucs-school-umc-diagnostic Version: 1.0.0-13A~4.4.0.202010021658 Branch: ucs_4.4-0 Scope: ucs-school-4.4
QA -> All OK -> VERIFY Changelog -> OK YAML -> OK
UCS@school 4.4 v7 has been released. https://docs.software-univention.de/changelog-ucsschool-4.4v7-de.html If this error occurs again, please clone this bug.