Menu

#137 Courier Mail support

Next_Release
closed-fixed
nobody
None
5
2017-06-06
2017-03-07
Lars Althof
No

Hi,
I made a first stab at adding support for Courier Mail. Please review and consider for adoption.

BR
Lars

2 Attachments

Discussion

  • Roland Gruber

    Roland Gruber - 2017-03-09

    Thank you very much! I will check the file the next few days.

     
  • Lars Althof

    Lars Althof - 2017-03-12

    I updated the module and added one more. I thought about it, but I really don't see that it makes sense to use profile for these, but I am open to suggestions.

     
  • Roland Gruber

    Roland Gruber - 2017-03-20

    Thanks, looks good. :)

    I guess the extension for users depends on inetOrgPerson and posixAccount? I see in schema that mail and homeDirectory is required?

     
  • Lars Althof

    Lars Althof - 2017-03-27

    Actually it shouldn't depend on posixAccount. I realized that too. I am working on adding the missing attributes and intelligence for when to show them.
    I did run into a problem. I added an option for profile and it works fine, but when I tried adding one for settings, I can't get it to show.

     
  • Roland Gruber

    Roland Gruber - 2017-04-10

    Do you need any help? I f yes please attach your latest version and what issue you have.

     
  • Lars Althof

    Lars Althof - 2017-04-10

    I have just been busy, but some advice on why I can't get the configuration options to show up would be nice.

     
  • Lars Althof

    Lars Althof - 2017-04-10

    Now with support for the two missing attributes when posixAccount is not present. At least I think so. Did you say that it will be possible with multiple configurations for Users in 5.8?

     
  • Lars Althof

    Lars Althof - 2017-04-11

    I think courierMailAccount is almost there now. courierMailAlias is giving me an error:

    Notice: Undefined index: or in /home/webs/resources/ldap-account-manager/lib/modules.inc on line 105 Notice: Undefined index: or in /home/webs/resources/ldap-account-manager/lib/modules.inc on line 108 Notice: Undefined index: and in /home/webs/resources/ldap-account-manager/lib/modules.inc on line 125 Notice: Undefined index: and in /home/webs/resources/ldap-account-manager/lib/modules.inc on line 126

     
  • Roland Gruber

    Roland Gruber - 2017-04-17

    Thanks a lot for the update! :)

    I did not get these errors on develop branch. Can you check with this?
    A possible issue might be that you should set a filter in get_metaData():

    // LDAP filter
    $return["ldap_filter"] = array('or' => "(objectClass=courierMailAlias)");

     
  • Lars Althof

    Lars Althof - 2017-04-17

    That did it.

    I am new to github, how do I check out a copy of the dev branch?

     
  • Roland Gruber

    Roland Gruber - 2017-04-17

    You can go here and select the "develop" branch:

    https://github.com/LDAPAccountManager/lam

    Then choose "Clone or download". In case you are not familiar with git you can simply download the zip. You find the same structure as in LAM's tar.bz2 downloads in lam subdirectory and can use this to run LAM.

     
  • Roland Gruber

    Roland Gruber - 2017-06-06
    • status: open --> accepted
     
  • Roland Gruber

    Roland Gruber - 2017-06-06

    Added to develop branch

     
  • Roland Gruber

    Roland Gruber - 2017-06-06
    • status: accepted --> closed-fixed