Bug 41115 - Adjust UCS@school LDAP ACL's
Adjust UCS@school LDAP ACL's
Status: CLOSED FIXED
Product: UCS@school
Classification: Unclassified
Component: LDAP
UCS@school 4.1
Other Linux
: P5 normal (vote)
: UCS@school 4.1 R2
Assigned To: Florian Best
Sönke Schwardt-Krummrich
:
: 25869 (view as bug list)
Depends on:
Blocks: 49827 41116 41720 42065 43042
  Show dependency treegraph
 
Reported: 2016-04-25 17:03 CEST by Florian Best
Modified: 2019-07-09 16:53 CEST (History)
2 users (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Florian Best univentionstaff 2016-04-25 17:03:09 CEST
The LDAP ACL's have to be adjusted for UCS@school to include the new attributes/options for the specific user roles.

1. All user containers should be replicated to all school DC's.
cn=(admins|lehrer|schueler|lehrer und mitarbeiter|mitarbeiter),cn=users,ou=…

2. All school DC's must be able to read all user objects which have ucsschoolSchool=$OU.

3. All current rules have to be adjusted to work with the new uscschoolSchool attribute.
Comment 1 Florian Best univentionstaff 2016-04-25 17:15:19 CEST
We should also make sure that this has no side effects as only a part of the OU structure is replicated. This might lead to noObject exceptions if trying to use the ucsschool-lib for some objects in a not completely replicated OU.
Comment 2 Florian Best univentionstaff 2016-04-25 17:28:15 CEST
In point 1. only the containers not the contents should be readable!
Comment 3 Florian Best univentionstaff 2016-05-11 11:12:36 CEST
4. The ACL's must allow to access cn=groups,$OU with scope=one to be able to resolve the (primary) groups of all users on the DC Slave.
Comment 4 Florian Best univentionstaff 2016-05-12 14:57:54 CEST
A closer look at svn r69292 should be done. All changes there should be documented.
Comment 5 Sönke Schwardt-Krummrich univentionstaff 2016-05-12 15:06:28 CEST
(In reply to Florian Best from comment #4)
> A closer look at svn r69292 should be done. All changes there should be
> documented.

Please add them to the YAML file.
Comment 6 Florian Best univentionstaff 2016-05-13 07:33:40 CEST
in svn r69291 I added a script ldap_acl_dump.py which writes an LDIF for every object it finds containing the permissions for each attribute.

e.g. 
dn: ou=oldschool,dc=school,dc=local
displayName: =rscxd
objectClass: =rscxd
ou: =rscxd
ucsschoolClassShareFileServer: =rscxd
ucsschoolHomeShareFileServer: =rscxd
univentionObjectType: =rscxd
univentionPolicyReference: =rscxd
Comment 7 Florian Best univentionstaff 2016-05-27 11:44:04 CEST
*** Bug 25869 has been marked as a duplicate of this bug. ***
Comment 8 Florian Best univentionstaff 2016-06-01 12:30:33 CEST
The ACL's have been adjusted.
cn=users, cn=examuser and cn=groups underneath of a UCS@school OU are replicated to all DC Slaves.
School DC's as well as school users can read every object belonging to one of the own schools.

ucs-school-ldap-acls-master (14.0.1-1):
r69691 | Bug #41115: whitespace cleanup
r69690 | Bug #41115: prevent read access to every object by all school objects
r69564 | Bug #41115: revert regression which caused failed.ldif during join
r69322 | Bug #41115: adjust LDAP ACL for new school structure
r69292 | Bug #41115: adjust broken/untidy ACL rules
r69291 | Bug #41115: adjust LDAP ACL for new school structure
r69265 | Bug #41115: adjust LDAP ACL for new school structure
r69247 | Bug #41115: simplify ACL logic, this seems to be unnecessary
r69246 | Bug #41115: adjust LDAP ACL for new school structure
r69245 | Bug #41115: remove unneeded rules
r69130 | Bug #41115: adjust LDAP ACL for new school structure
r69129 | Bug #41115: adjust LDAP ACL for new school structure
r69089 | Bug #41115: adjust LDAP ACL for new school structure
r69088 | Bug #41115: preserve permissions instead of dropping them
r69087 | Bug #41115: revert last commit
r68899 | Bug #41115: start LDAP ACL adjustment to use ucsschoolSchool attribute
Comment 9 Sönke Schwardt-Krummrich univentionstaff 2016-07-03 23:23:17 CEST
Bug 41720 has been opened for a minor/medium issue.
Everything else looked ok during manual comparison. See Bug 41116.
Comment 10 Florian Best univentionstaff 2016-07-04 16:55:59 CEST
UCS@school 4.1 R2 has been released:
http://docs.software-univention.de/release-notes-ucsschool-4.1R2v1-de.pdf

If this error occurs again, please use "Clone This Bug".