Bug 17994 - Berechtigungen fehlen wenn "Mail: IMAP Ordner" ohne E-Mail-Adresse angelegt wird
Berechtigungen fehlen wenn "Mail: IMAP Ordner" ohne E-Mail-Adresse angelegt wird
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: Mail
UCS 2.3
Other Linux
: P5 normal (vote)
: UCS 3.1
Assigned To: Felix Botner
Sönke Schwardt-Krummrich
: interim-1
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-03-29 12:22 CEST by Janek Walkenhorst
Modified: 2012-12-12 21:10 CET (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 Janek Walkenhorst univentionstaff 2010-03-29 12:22:25 CEST
Wenn ein "Mail: IMAP Ordner" ohne eine primäre E-Mail-Adresse angelegt wird, werden beim nachträglichen Eintragen der Adresse die Berechtigungen nicht gesetzt:

 shared/foo5@univention.qa:

richtig wäre es so:

 shared/foo4@univention.qa:
   anyone p


Es sollte auch geprüft werden, ob beim Ändern der E-Mail-Adresse die Berechtigungen korrekt geändert/übernommen werden.
Comment 1 Felix Botner univentionstaff 2012-06-07 18:01:22 CEST
Das klappt zwar nun, aber wenn man einen Folder ohne primäre Adresse anlegt (macht das überhaupt Sinn?) sehen die ACL's so aus

-> lam shared/folder4@ff.gg                                    
anyone ed

Wenn den Folder mit primäre Adresse anlegt so:

-> lam shared/folder3@ff.gg                                    
anyone p

Ich glaub ersteres ist falsch und keine richtige ACL. Das sollte man sich nochmal anschauen.
Comment 2 Stefan Gohmann univentionstaff 2012-07-17 17:09:30 CEST
UCS 3.1 will be the next release.
Comment 3 Felix Botner univentionstaff 2012-07-24 14:06:30 CEST
Im listener cyrus-shared-folder.py wurde die Policy aus den univentionMailACL's immer um 'p' (p -> cyrus acl zum posten) erweitert. Wenn aber das Recht "kein Zugriff" ausgewählt wurde, hat er an die ACL "none" das "p" angehängt und mit diesem "nonep" dann die ACL über cyradm gesetzt. Dabei kam dann so etwas raus wie anyone ed oder anyone ped.

Das fix_anyone_acl aus dem Listener wurde entfernt. Statt dessen wird in setacl bei Policies für "anyone" die Policy um "p" erweitert bzw. aus "p" gesetzt wenn die Policy "none" ist.

Changelog Eintrag vorhanden.
Comment 4 Sönke Schwardt-Krummrich univentionstaff 2012-09-12 18:30:52 CEST
(In reply to comment #3)
> Im listener cyrus-shared-folder.py wurde die Policy aus den univentionMailACL's
> immer um 'p' (p -> cyrus acl zum posten) erweitert. Wenn aber das Recht "kein
> Zugriff" ausgewählt wurde, hat er an die ACL "none" das "p" angehängt und mit
> diesem "nonep" dann die ACL über cyradm gesetzt. Dabei kam dann so etwas raus
> wie anyone ed oder anyone ped.
> 
> Das fix_anyone_acl aus dem Listener wurde entfernt. Statt dessen wird in setacl
> bei Policies für "anyone" die Policy um "p" erweitert bzw. aus "p" gesetzt wenn
> die Policy "none" ist.

→ REOPEN
Vor der Änderung wurde versucht, das Postflag "p" nur dann zu setzen, wenn wirklich eine primäre Mailadresse hinterlegt war. Jetzt wird das Flag immer gesetzt, was alle IMAP-Benutzer dazu befähigt, Mails in dem Shared Folder abzulegen. Shared Folder ohne Mailadresse, auf die wirklich nur ein eingeschränkter Benutzerkreis Lese- & Schreibrechte besitzt, sind somit nicht mehr möglich.
Bitte beachten, dass ggf. beim Setzen/Entfernen der primären Mailadresse die ACLs neu generiert werden müssen.
Comment 5 Felix Botner univentionstaff 2012-09-13 13:12:39 CEST
p sollte jetzt nur noch für Folder mit mailPrimaryAddress gesetzt werden, sonst wird "none" verwendet.

Die Default ACL werden beim Anlegen bzw. nachträglichem Hinzufügen der mailPrimaryAddress und bei Änderungen des Objekts gesetzt.
Comment 6 Sönke Schwardt-Krummrich univentionstaff 2012-09-14 11:08:41 CEST
(In reply to comment #5)
> p sollte jetzt nur noch für Folder mit mailPrimaryAddress gesetzt werden, sonst
> wird "none" verwendet.
> 
> Die Default ACL werden beim Anlegen bzw. nachträglichem Hinzufügen der
> mailPrimaryAddress und bei Änderungen des Objekts gesetzt.

"anyone" bekommt jetzt ein "p" gesetzt, wenn eine primäre Mailadresse gesetzt ist und das Flag wird wieder entfernt, wenn die Mailadresse entfernt wird. Die beeinflusst nicht die ACLs von anderen Benutzern, die ggf. zusätzlich gesetzt sind.
→ VERIFIED
Comment 7 Stefan Gohmann univentionstaff 2012-12-12 21:10:34 CET
UCS 3.1-0 has been released: 
 http://forum.univention.de/viewtopic.php?f=54&t=2125

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