OBM
  1. OBM
  2. OBMFULL-6434

T#14286 Can't include/exclude shared mailboxes in imap-archive

    Details

    • Story Points:
      9
    • Similar Issues:
      Show 9 results 
    • Rank:
      6911
    • Sprint:
      2016#01

      Description

      Reproduction steps

      1. In obm ui, create a shared mailbox (let's call it "shared1").
      2. In obm ui, go to the imap-archive page, an try to include/exclude "shared1" mailbox.

      Current behavior

      1. The shared mailbox is not found. This textbox only matches users mailboxes NOK

      Expected behavior

      1. This field should allow to select users mailboxes and shared mailboxes. OK

      Additional information

      1. Shared mailboxes being seen by cyrus as regular mailboxes, those are being archived by default (for example, if everything is archived except a specific user mailbox, the shared mailboxes will be archived).

        Activity

        Hide
        Stephane COLSON added a comment -

        OK, no more logs and everything is archived (tested with a few emails in a shared mailbox)

        Only tested on Debian 7

        Show
        Stephane COLSON added a comment - OK, no more logs and everything is archived (tested with a few emails in a shared mailbox) Only tested on Debian 7
        Hide
        Jenkins Continuous Integration Server added a comment -

        FAILURE: Integrated in poll-master #241
        OBMFULL-6434 Don't use GETACL when creating users temporary folder (aduprat: 0ae38bafb6cb37192c5e0a4a9350235e9c971115)

        • java/sync/imap-archive/src/main/java/org/obm/imap/archive/services/MailboxProcessing.java
        • java/sync/imap-archive/src/main/java/org/obm/imap/archive/services/DryMailboxProcessing.java
        • java/sync/imap-archive/src/test/java/org/obm/imap/archive/services/ImapArchiveProcessingTest.java
        Show
        Jenkins Continuous Integration Server added a comment - FAILURE: Integrated in poll-master #241 OBMFULL-6434 Don't use GETACL when creating users temporary folder (aduprat: 0ae38bafb6cb37192c5e0a4a9350235e9c971115 ) java/sync/imap-archive/src/main/java/org/obm/imap/archive/services/MailboxProcessing.java java/sync/imap-archive/src/main/java/org/obm/imap/archive/services/DryMailboxProcessing.java java/sync/imap-archive/src/test/java/org/obm/imap/archive/services/ImapArchiveProcessingTest.java
        Hide
        Jenkins Continuous Integration Server added a comment -

        SUCCESS: Integrated in poll-master #243
        OBMFULL-6434 GetACLCommand may handle folders with spaces (aduprat: d94a004daa6360e7449723cb1881b069cf3dae72)

        • java/sync/linagora-imap/src/test/java/org/obm/push/minig/imap/command/GetACLCommandTest.java
        • java/sync/linagora-imap/src/main/java/org/obm/push/minig/imap/command/GetACLCommand.java
          OBMFULL-6434 Override existing rights for Cyrus (aduprat: 5e0a94b530fa31155f9ce675ad317380705e43b7)
        • java/sync/imap-archive/src/main/java/org/obm/imap/archive/services/MailboxProcessing.java
        Show
        Jenkins Continuous Integration Server added a comment - SUCCESS: Integrated in poll-master #243 OBMFULL-6434 GetACLCommand may handle folders with spaces (aduprat: d94a004daa6360e7449723cb1881b069cf3dae72 ) java/sync/linagora-imap/src/test/java/org/obm/push/minig/imap/command/GetACLCommandTest.java java/sync/linagora-imap/src/main/java/org/obm/push/minig/imap/command/GetACLCommand.java OBMFULL-6434 Override existing rights for Cyrus (aduprat: 5e0a94b530fa31155f9ce675ad317380705e43b7 ) java/sync/imap-archive/src/main/java/org/obm/imap/archive/services/MailboxProcessing.java
        Hide
        Antoine DUPRAT added a comment -

        QA on production leads to following modifications:

        • regression on users processing: don't use GETACL when creating temporary folders in this case
        • GETACL imap command has a bug when a folder contains a space character
        • In some configuration, Cyrus rights are lost on the mailboxes created by the process: enforce them

        Those three points are fixed and checked on the production.

        Show
        Antoine DUPRAT added a comment - QA on production leads to following modifications: regression on users processing: don't use GETACL when creating temporary folders in this case GETACL imap command has a bug when a folder contains a space character In some configuration, Cyrus rights are lost on the mailboxes created by the process: enforce them Those three points are fixed and checked on the production.
        Hide
        Stephane COLSON added a comment -

        OK, tested with 3.2.0~0.rc3, with folders and sub-folders with or without spaces for users mailbox or shared mailbox.

        Show
        Stephane COLSON added a comment - OK, tested with 3.2.0~0.rc3, with folders and sub-folders with or without spaces for users mailbox or shared mailbox.

          People

          • Assignee:
            Antoine DUPRAT
            Reporter:
            Lionel MOREIRA
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Agile