Francis Augusto Medeiros-Logeay via FreeIPA-users wrote: > Hi, > > I have almost finished a plugin for FreeIPA, so that admins can have > similar functionality found on Postfix Admin. > > freeipa-postfixadmin.png > freeipa-postfixadmin/README.md at main · oculos/freeipa-postfixadmin > <https://github.com/oculos/freeipa-postfixadmin/blob/main/README.md> > github.com > <https://github.com/oculos/freeipa-postfixadmin/blob/main/README.md> > > > There is already a good plugin that does a bit of that, but the goal is > a bit different. My main goal is not to mix up postfix configuration > with groups and hosts, but have separate entities for domain, aliases > and virtual domains, in addition to mailboxes. > > It was written mostly to allow me to migrate my mailboxes from MySQL to > FreeIPA, and I don’t have a huge postfix configuration - I only have > multiple domains, mailboxes, aliases and virtual domains, so that’s the > functionality I wanted with this plugin. > > There are a few things missing before this can go in production > («production» here means to actually migrate my mailboxes to FreeIPA), > adding a mailbox to ipa users on the gui being the most important one. > > I would appreciate any comments and feedbacks regarding this plugin. It > wasn’t easy to understand the logic on how to write one, but I got the > hang of it (for simple stuff).
Looks like a good start! I have a few questions: 1. Where did the 1.3.6.1.4.1.60062 prefix for objectclasses and attributes come from? Is this a private one? 2. The domain_mod has a pre_callback that validates the domain. Isn't it too late at this point? In any case, you may be able to move the validator to the Param instead. 3. If you specify all the objectclasses at in object_class you don't need to manually add them. 4. There is a helper you can use to determine if an objectclass is already in an entry, self.obj.has_objectclass() 5. If you convert create_default_objects.ldif to an update file you can use $SUFFIX instead of hardcoding the domain part. 6. I'd rename create_attributes.ldif to something that looks more like a schema file. By naming it is very clear but it's convention. rob _______________________________________________ FreeIPA-users mailing list -- [email protected] To unsubscribe send an email to [email protected] Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/[email protected] Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
