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...
ClosedPublic

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

Details

Summary

..."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

none

Diff Detail

Repository
rP pykolab
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
adomaitis created this revision.Mar 12 2018, 1:56 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.