OBM
  1. OBM
  2. OBMFULL-6447

OBM-provisionning does not honor userMailboxDefaultFolders

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.1.6
    • Fix Version/s: 3.2.0
    • Component/s: provisioning-api
    • Labels:
      None
    • Story Points:
      3
    • Similar Issues:
      Show 9 results 
    • Rank:
      6942
    • Sprint:
      2016#01

      Description

      Pre-conditions

      Reproduction steps

      1. edit /etc/obm/obm_conf.ini file on the obm-provisionning server and change the key userMailboxDefaultFolders as bellow:
      userMailboxDefaultFolders = 'Trash,Drafts,Junk,Templates,Sent'

      The nrun the papi command to create user

      ./createUser.sh <UUID-DU-DOMAINE> user1

      Current behavior

      1. mailbox is created with a SPAM folder instead of the Junk folder specified in obm_conf.ini

      Expected behavior

      1. mailbox should be created with a Junk folder as specified in obm_conf.ini

      Additional informations

      The case the variable is not defined in the ini file should not trigger an error but create only an inbox without subfolders (as if the variable were empty)

        Activity

        Hide
        David Dolcimascolo added a comment -

        Please do a test with accented mailbox names.

        Show
        David Dolcimascolo added a comment - Please do a test with accented mailbox names.
        Hide
        David Dolcimascolo added a comment -

        To QA:

        • Impacts obm-provisioning
          • The parameter userMailboxDefaultFolders of obm_conf.ini is now taken into account.
        • To test, play with this setting and create users through PAPI. Check everytime that IMAP folders are correctly created.

        Tested on CentOS7 with papi-cli.

        Show
        David Dolcimascolo added a comment - To QA: Impacts obm-provisioning The parameter userMailboxDefaultFolders of obm_conf.ini is now taken into account. To test, play with this setting and create users through PAPI. Check everytime that IMAP folders are correctly created. Tested on CentOS7 with papi-cli.
        Hide
        Jenkins Continuous Integration Server added a comment -

        SUCCESS: Integrated in poll-master #221
        OBMFULL-6447 Moved removeEnclosingQuotes to IniFile. (ddolcimascolo: 9fb34edf633b543d30a61f8889fd4c74a974a74a)

        • java/sync/configuration/src/main/java/org/obm/configuration/utils/IniFile.java
        • java/sync/configuration/src/test/java/org/obm/configuration/DatabaseConfigurationImplTest.java
        • java/sync/configuration/src/main/java/org/obm/configuration/DatabaseConfigurationImpl.java
        • java/sync/configuration/src/test/java/org/obm/configuration/utils/IniFileTest.java
          OBMFULL-6447 Added getUserMailboxDefaultFolders to ConfigurationService. (ddolcimascolo: 250768aad08e1f6e8fd1c7130365d3c8bc6db32e)
        • java/sync/configuration/src/test/java/org/obm/configuration/ConfigurationServiceImplTest.java
        • java/sync/configuration-test/src/main/java/org/obm/StaticConfigurationService.java
        • java/sync/configuration/src/main/java/org/obm/configuration/ConfigurationServiceImpl.java
        • java/sync/configuration/src/main/java/org/obm/configuration/ConfigurationService.java
        • java/sync/configuration-test/src/main/java/org/obm/Configuration.java
          OBMFULL-6447 obm-provisioning now honors userMailboxDefaultFolders. (ddolcimascolo: 7eec7091e40e667eef0f52129c6876ec03ed5b36)
        • java/sync/cyrus-client/src/main/java/org/obm/cyrus/imap/admin/Connection.java
        • java/sync/cyrus-client/src/main/java/org/obm/cyrus/imap/admin/CyrusManagerImpl.java
        • java/sync/cyrus-client/src/test/java/org/obm/cyrus/imap/admin/ConnectionImplTest.java
        • java/sync/cyrus-client/src/test/java/org/obm/cyrus/imap/admin/CyrusManagerImplTest.java
        • java/sync/cyrus-client/src/main/java/org/obm/cyrus/imap/admin/ConnectionImpl.java
        Show
        Jenkins Continuous Integration Server added a comment - SUCCESS: Integrated in poll-master #221 OBMFULL-6447 Moved removeEnclosingQuotes to IniFile. (ddolcimascolo: 9fb34edf633b543d30a61f8889fd4c74a974a74a ) java/sync/configuration/src/main/java/org/obm/configuration/utils/IniFile.java java/sync/configuration/src/test/java/org/obm/configuration/DatabaseConfigurationImplTest.java java/sync/configuration/src/main/java/org/obm/configuration/DatabaseConfigurationImpl.java java/sync/configuration/src/test/java/org/obm/configuration/utils/IniFileTest.java OBMFULL-6447 Added getUserMailboxDefaultFolders to ConfigurationService. (ddolcimascolo: 250768aad08e1f6e8fd1c7130365d3c8bc6db32e ) java/sync/configuration/src/test/java/org/obm/configuration/ConfigurationServiceImplTest.java java/sync/configuration-test/src/main/java/org/obm/StaticConfigurationService.java java/sync/configuration/src/main/java/org/obm/configuration/ConfigurationServiceImpl.java java/sync/configuration/src/main/java/org/obm/configuration/ConfigurationService.java java/sync/configuration-test/src/main/java/org/obm/Configuration.java OBMFULL-6447 obm-provisioning now honors userMailboxDefaultFolders. (ddolcimascolo: 7eec7091e40e667eef0f52129c6876ec03ed5b36 ) java/sync/cyrus-client/src/main/java/org/obm/cyrus/imap/admin/Connection.java java/sync/cyrus-client/src/main/java/org/obm/cyrus/imap/admin/CyrusManagerImpl.java java/sync/cyrus-client/src/test/java/org/obm/cyrus/imap/admin/ConnectionImplTest.java java/sync/cyrus-client/src/test/java/org/obm/cyrus/imap/admin/CyrusManagerImplTest.java java/sync/cyrus-client/src/main/java/org/obm/cyrus/imap/admin/ConnectionImpl.java
        Hide
        Stephane COLSON added a comment -

        OK with version 3.2.0~0.alpha0+git20160419-1747-3fc3c43 on Debian
        Tested with papi and

        • "userMailboxDefaultFolders = "Envoyés,Corbeille,Brouillons,Eléments indésirables" => fields with accents names are ignored with a error log in obm-provisioning.log
        • Any other folder is created, also if you use what is suggested as a comment in the ini file "userMailboxDefaultFolders = "Envoyés,Corbeille,Brouillons,Eléments indésirables""
        Show
        Stephane COLSON added a comment - OK with version 3.2.0~0.alpha0+git20160419-1747-3fc3c43 on Debian Tested with papi and "userMailboxDefaultFolders = "Envoyés,Corbeille,Brouillons,Eléments indésirables" => fields with accents names are ignored with a error log in obm-provisioning.log Any other folder is created, also if you use what is suggested as a comment in the ini file "userMailboxDefaultFolders = "Envoyés,Corbeille,Brouillons,Eléments indésirables""

          People

          • Assignee:
            David Dolcimascolo
            Reporter:
            Alexandre CHAPELLON
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Agile