Page MenuHomePhorge

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.
Tags
None
Referenced Files
F11588388: D571.id.diff
Thu, Mar 28, 8:01 PM
Unknown Object (File)
Mon, Mar 25, 2:50 AM
Unknown Object (File)
Sun, Mar 24, 6:12 PM
Unknown Object (File)
Fri, Mar 1, 5:48 AM
Unknown Object (File)
Feb 15 2024, 5:02 AM
Unknown Object (File)
Feb 9 2024, 2:23 AM
Unknown Object (File)
Feb 5 2024, 2:00 AM
Unknown Object (File)
Jan 30 2024, 11:40 AM

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
Branch
T80958
Lint
Lint Skipped
Unit
No Test Coverage
Build Status
Buildable 16885
Build 9346: arc lint + arc unit

Event Timeline

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

  • To avoid double user/ prefix, check if user_mailbox_exists only by result_attribute
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.