In some cases kolabd fails to cleanly update IMAP mailbox after LDAP changes. That is usual when result_attribute value is in upper case. Mailbox modification seems to follow slightly different code path, by shortcutting to IMAP pykolab function...

Authored by adomaitis on Mar 12 2018, 1:56 PM.



..."has_folder()" instead of going via "user_mailbox_exists()", which has a code to downcase mailbox name.

Do not shortcut to "has_folder()" function, use "user_mailbox_exists()" to downcase mailbox name before checking if it exists.

Test Plan


Diff Detail

rP pykolab
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

adomaitis created this revision.Mar 12 2018, 1:56 PM
machniak edited edge metadata.Mar 12 2018, 2:05 PM

Looks to me like now you'll add user/ prefix twice. See in user_mailbox_extsts() and code above your change.

adomaitis updated this revision to Diff 1291.Mar 12 2018, 2:19 PM
  • To avoid double user/ prefix, check if user_mailbox_exists only by result_attribute
machniak accepted this revision.Mar 12 2018, 2:54 PM
This revision is now accepted and ready to land.Mar 12 2018, 2:54 PM
This revision was automatically updated to reflect the committed changes.