Opush
  1. Opush
  2. OP-217

Synchronization is blocked when an ICS invitation has no DTSTAMP

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Normal Normal
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.1.1
    • Labels:
      None
    • Rank:
      6873

      Description

      Pre-conditions

      usera has an email containing an ICS without DTSTAMP in its INBOX (even is it's not legal against the spec)
      usera has another regular email

      Reproduction steps

      1. As usera synchronize its INBOX

      Current behavior

      1. NOK an exception is triggered in logs
      2. NOK no mail appear on its smartphone

      Expected behavior

      1. OK no exception appears in logs
      2. OK its emails appear on its smartphone
      3. OK the email with the invitation has a "created date" at 1970-01-01

      Additional informations

      4:15:19.313 [ERROR] {org.obm.push.handler.SyncHandler} [t40824/cSync/r38036/user@linagora.com] The field dtStamp is required
      java.lang.NullPointerException: The field dtStamp is required
      	at com.google.common.base.Preconditions.checkNotNull(Preconditions.java:229) ~[guava-16.0.jar:na]
      	at org.obm.push.bean.msmeetingrequest.MSMeetingRequest$Builder.build(MSMeetingRequest.java:185) ~[push-bean-3.1.0-rc3.jar:na]
      	at org.obm.push.mail.ICalendarConverter.convertToMSMeetingRequest(ICalendarConverter.java:127) ~[push-mail-3.1.0-rc3.jar:na]
      	at org.obm.push.mail.MailViewToMSEmailConverterImpl.convertICalendar(MailViewToMSEmailConverterImpl.java:209) ~[push-mail-3.1.0-rc3.jar:na]
      	at org.obm.push.mail.MailViewToMSEmailConverterImpl.convert(MailViewToMSEmailConverterImpl.java:99) ~[push-mail-3.1.0-rc3.jar:na]
      	at org.obm.push.mail.MSEmailFetcher.fetch(MSEmailFetcher.java:87) ~[push-mail-3.1.0-rc3.jar:na]
      	at org.obm.push.mail.EmailChangesFetcherImpl.fetchMSEmails(EmailChangesFetcherImpl.java:150) ~[push-mail-3.1.0-rc3.jar:na]
      	at org.obm.push.mail.EmailChangesFetcherImpl.emailAdditions(EmailChangesFetcherImpl.java:104) ~[push-mail-3.1.0-rc3.jar:na]
      	at org.obm.push.mail.EmailChangesFetcherImpl.fetch(EmailChangesFetcherImpl.java:77) ~[push-mail-3.1.0-rc3.jar:na]
      	at org.obm.push.mail.MailBackendImpl.fetchChanges(MailBackendImpl.java:291) ~[push-mail-3.1.0-rc3.jar:na]
      	at org.obm.push.mail.MailBackendImpl.continueWindowing(MailBackendImpl.java:283) ~[push-mail-3.1.0-rc3.jar:na]
      	at org.obm.push.mail.MailBackendImpl.getChanged(MailBackendImpl.java:256) ~[push-mail-3.1.0-rc3.jar:na]
      	at org.obm.breakdownduration.BreakdownDurationInterceptor.invoke(BreakdownDurationInterceptor.java:58) ~[breakdown-duration-3.1.2.jar:na]
      	at org.obm.push.ContentsExporter.getChanged(ContentsExporter.java:82) ~[push-core-3.1.0-rc3.jar:na]
      	at org.obm.push.handler.SyncHandler.retreiveServerModifications(SyncHandler.java:288) ~[push-core-3.1.0-rc3.jar:na]
      	at org.obm.push.handler.SyncHandler.doUpdates(SyncHandler.java:271) ~[push-core-3.1.0-rc3.jar:na]
      

        Activity

        Thomas HILAIRE made changes -
        Field Original Value New Value
        Description h4. Pre-conditions
        _usera_ has an email containing an ICS without DTSTAMP in its INBOX (even is it's not legal against the spec)
        _usera_ has another regular email

        h4. Reproduction steps
        # As _usera_ synchronize its INBOX

        h4. Current behavior
        # {color:red}NOK{color} an exception is triggered in logs
        # {color:red}NOK{color} no mail appear on its smartphone

        h4. Expected behavior
        # {color:green}OK{color} no exception appears in logs
        # {color:green}OK{color} its emails appear on its smartphone
        # {color:green}OK{color} the email with the invitation has a "created date" at 1970-01-01

        h4. Additional informations
        h4. Pre-conditions
        _usera_ has an email containing an ICS without DTSTAMP in its INBOX (even is it's not legal against the spec)
        _usera_ has another regular email

        h4. Reproduction steps
        # As _usera_ synchronize its INBOX

        h4. Current behavior
        # {color:red}NOK{color} an exception is triggered in logs
        # {color:red}NOK{color} no mail appear on its smartphone

        h4. Expected behavior
        # {color:green}OK{color} no exception appears in logs
        # {color:green}OK{color} its emails appear on its smartphone
        # {color:green}OK{color} the email with the invitation has a "created date" at 1970-01-01

        h4. Additional informations
        {code}
        4:15:19.313 [ERROR] {org.obm.push.handler.SyncHandler} [t40824/cSync/r38036/user@linagora.com] The field dtStamp is required
        java.lang.NullPointerException: The field dtStamp is required
        at com.google.common.base.Preconditions.checkNotNull(Preconditions.java:229) ~[guava-16.0.jar:na]
        at org.obm.push.bean.msmeetingrequest.MSMeetingRequest$Builder.build(MSMeetingRequest.java:185) ~[push-bean-3.1.0-rc3.jar:na]
        at org.obm.push.mail.ICalendarConverter.convertToMSMeetingRequest(ICalendarConverter.java:127) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MailViewToMSEmailConverterImpl.convertICalendar(MailViewToMSEmailConverterImpl.java:209) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MailViewToMSEmailConverterImpl.convert(MailViewToMSEmailConverterImpl.java:99) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MSEmailFetcher.fetch(MSEmailFetcher.java:87) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.EmailChangesFetcherImpl.fetchMSEmails(EmailChangesFetcherImpl.java:150) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.EmailChangesFetcherImpl.emailAdditions(EmailChangesFetcherImpl.java:104) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.EmailChangesFetcherImpl.fetch(EmailChangesFetcherImpl.java:77) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MailBackendImpl.fetchChanges(MailBackendImpl.java:291) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MailBackendImpl.continueWindowing(MailBackendImpl.java:283) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MailBackendImpl.getChanged(MailBackendImpl.java:256) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.breakdownduration.BreakdownDurationInterceptor.invoke(BreakdownDurationInterceptor.java:58) ~[breakdown-duration-3.1.2.jar:na]
        at org.obm.push.ContentsExporter.getChanged(ContentsExporter.java:82) ~[push-core-3.1.0-rc3.jar:na]
        at org.obm.push.handler.SyncHandler.retreiveServerModifications(SyncHandler.java:288) ~[push-core-3.1.0-rc3.jar:na]
        at org.obm.push.handler.SyncHandler.doUpdates(SyncHandler.java:271) ~[push-core-3.1.0-rc3.jar:na]
        {code}
        Thomas HILAIRE made changes -
        Description h4. Pre-conditions
        _usera_ has an email containing an ICS without DTSTAMP in its INBOX (even is it's not legal against the spec)
        _usera_ has another regular email

        h4. Reproduction steps
        # As _usera_ synchronize its INBOX

        h4. Current behavior
        # {color:red}NOK{color} an exception is triggered in logs
        # {color:red}NOK{color} no mail appear on its smartphone

        h4. Expected behavior
        # {color:green}OK{color} no exception appears in logs
        # {color:green}OK{color} its emails appear on its smartphone
        # {color:green}OK{color} the email with the invitation has a "created date" at 1970-01-01

        h4. Additional informations
        {code}
        4:15:19.313 [ERROR] {org.obm.push.handler.SyncHandler} [t40824/cSync/r38036/user@linagora.com] The field dtStamp is required
        java.lang.NullPointerException: The field dtStamp is required
        at com.google.common.base.Preconditions.checkNotNull(Preconditions.java:229) ~[guava-16.0.jar:na]
        at org.obm.push.bean.msmeetingrequest.MSMeetingRequest$Builder.build(MSMeetingRequest.java:185) ~[push-bean-3.1.0-rc3.jar:na]
        at org.obm.push.mail.ICalendarConverter.convertToMSMeetingRequest(ICalendarConverter.java:127) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MailViewToMSEmailConverterImpl.convertICalendar(MailViewToMSEmailConverterImpl.java:209) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MailViewToMSEmailConverterImpl.convert(MailViewToMSEmailConverterImpl.java:99) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MSEmailFetcher.fetch(MSEmailFetcher.java:87) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.EmailChangesFetcherImpl.fetchMSEmails(EmailChangesFetcherImpl.java:150) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.EmailChangesFetcherImpl.emailAdditions(EmailChangesFetcherImpl.java:104) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.EmailChangesFetcherImpl.fetch(EmailChangesFetcherImpl.java:77) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MailBackendImpl.fetchChanges(MailBackendImpl.java:291) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MailBackendImpl.continueWindowing(MailBackendImpl.java:283) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MailBackendImpl.getChanged(MailBackendImpl.java:256) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.breakdownduration.BreakdownDurationInterceptor.invoke(BreakdownDurationInterceptor.java:58) ~[breakdown-duration-3.1.2.jar:na]
        at org.obm.push.ContentsExporter.getChanged(ContentsExporter.java:82) ~[push-core-3.1.0-rc3.jar:na]
        at org.obm.push.handler.SyncHandler.retreiveServerModifications(SyncHandler.java:288) ~[push-core-3.1.0-rc3.jar:na]
        at org.obm.push.handler.SyncHandler.doUpdates(SyncHandler.java:271) ~[push-core-3.1.0-rc3.jar:na]
        {code}
        eh4. Pre-conditions
        _usera_ has an email containing an ICS without DTSTAMP in its INBOX (even is it's not legal against the spec)
        _usera_ has another regular email

        h4. Reproduction steps
        # As _usera_ synchronize its INBOX

        h4. Current behavior
        # {color:red}NOK{color} an exception is triggered in logs
        # {color:red}NOK{color} no mail appear on its smartphone

        h4. Expected behavior
        # {color:green}OK{color} no exception appears in logs
        # {color:green}OK{color} its emails appear on its smartphone
        # {color:green}OK{color} the email with the invitation has a "created date" at 1970-01-01

        h4. Additional informations
        {code}
        4:15:19.313 [ERROR] {org.obm.push.handler.SyncHandler} [t40824/cSync/r38036/user@linagora.com] The field dtStamp is required
        java.lang.NullPointerException: The field dtStamp is required
        at com.google.common.base.Preconditions.checkNotNull(Preconditions.java:229) ~[guava-16.0.jar:na]
        at org.obm.push.bean.msmeetingrequest.MSMeetingRequest$Builder.build(MSMeetingRequest.java:185) ~[push-bean-3.1.0-rc3.jar:na]
        at org.obm.push.mail.ICalendarConverter.convertToMSMeetingRequest(ICalendarConverter.java:127) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MailViewToMSEmailConverterImpl.convertICalendar(MailViewToMSEmailConverterImpl.java:209) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MailViewToMSEmailConverterImpl.convert(MailViewToMSEmailConverterImpl.java:99) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MSEmailFetcher.fetch(MSEmailFetcher.java:87) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.EmailChangesFetcherImpl.fetchMSEmails(EmailChangesFetcherImpl.java:150) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.EmailChangesFetcherImpl.emailAdditions(EmailChangesFetcherImpl.java:104) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.EmailChangesFetcherImpl.fetch(EmailChangesFetcherImpl.java:77) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MailBackendImpl.fetchChanges(MailBackendImpl.java:291) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MailBackendImpl.continueWindowing(MailBackendImpl.java:283) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MailBackendImpl.getChanged(MailBackendImpl.java:256) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.breakdownduration.BreakdownDurationInterceptor.invoke(BreakdownDurationInterceptor.java:58) ~[breakdown-duration-3.1.2.jar:na]
        at org.obm.push.ContentsExporter.getChanged(ContentsExporter.java:82) ~[push-core-3.1.0-rc3.jar:na]
        at org.obm.push.handler.SyncHandler.retreiveServerModifications(SyncHandler.java:288) ~[push-core-3.1.0-rc3.jar:na]
        at org.obm.push.handler.SyncHandler.doUpdates(SyncHandler.java:271) ~[push-core-3.1.0-rc3.jar:na]
        {code}
        Thomas HILAIRE made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Thomas HILAIRE made changes -
        Status In Progress [ 3 ] In Review [ 10001 ]
        Thomas HILAIRE made changes -
        Status In Review [ 10001 ] Ready To Merge [ 10002 ]
        Resolution Fixed [ 1 ]
        Thomas HILAIRE made changes -
        Status Ready To Merge [ 10002 ] Resolved [ 5 ]
        Stephane COLSON made changes -
        Description eh4. Pre-conditions
        _usera_ has an email containing an ICS without DTSTAMP in its INBOX (even is it's not legal against the spec)
        _usera_ has another regular email

        h4. Reproduction steps
        # As _usera_ synchronize its INBOX

        h4. Current behavior
        # {color:red}NOK{color} an exception is triggered in logs
        # {color:red}NOK{color} no mail appear on its smartphone

        h4. Expected behavior
        # {color:green}OK{color} no exception appears in logs
        # {color:green}OK{color} its emails appear on its smartphone
        # {color:green}OK{color} the email with the invitation has a "created date" at 1970-01-01

        h4. Additional informations
        {code}
        4:15:19.313 [ERROR] {org.obm.push.handler.SyncHandler} [t40824/cSync/r38036/user@linagora.com] The field dtStamp is required
        java.lang.NullPointerException: The field dtStamp is required
        at com.google.common.base.Preconditions.checkNotNull(Preconditions.java:229) ~[guava-16.0.jar:na]
        at org.obm.push.bean.msmeetingrequest.MSMeetingRequest$Builder.build(MSMeetingRequest.java:185) ~[push-bean-3.1.0-rc3.jar:na]
        at org.obm.push.mail.ICalendarConverter.convertToMSMeetingRequest(ICalendarConverter.java:127) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MailViewToMSEmailConverterImpl.convertICalendar(MailViewToMSEmailConverterImpl.java:209) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MailViewToMSEmailConverterImpl.convert(MailViewToMSEmailConverterImpl.java:99) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MSEmailFetcher.fetch(MSEmailFetcher.java:87) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.EmailChangesFetcherImpl.fetchMSEmails(EmailChangesFetcherImpl.java:150) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.EmailChangesFetcherImpl.emailAdditions(EmailChangesFetcherImpl.java:104) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.EmailChangesFetcherImpl.fetch(EmailChangesFetcherImpl.java:77) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MailBackendImpl.fetchChanges(MailBackendImpl.java:291) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MailBackendImpl.continueWindowing(MailBackendImpl.java:283) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MailBackendImpl.getChanged(MailBackendImpl.java:256) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.breakdownduration.BreakdownDurationInterceptor.invoke(BreakdownDurationInterceptor.java:58) ~[breakdown-duration-3.1.2.jar:na]
        at org.obm.push.ContentsExporter.getChanged(ContentsExporter.java:82) ~[push-core-3.1.0-rc3.jar:na]
        at org.obm.push.handler.SyncHandler.retreiveServerModifications(SyncHandler.java:288) ~[push-core-3.1.0-rc3.jar:na]
        at org.obm.push.handler.SyncHandler.doUpdates(SyncHandler.java:271) ~[push-core-3.1.0-rc3.jar:na]
        {code}
        h4. Pre-conditions
        _usera_ has an email containing an ICS without DTSTAMP in its INBOX (even is it's not legal against the spec)
        _usera_ has another regular email

        h4. Reproduction steps
        # As _usera_ synchronize its INBOX

        h4. Current behavior
        # {color:red}NOK{color} an exception is triggered in logs
        # {color:red}NOK{color} no mail appear on its smartphone

        h4. Expected behavior
        # {color:green}OK{color} no exception appears in logs
        # {color:green}OK{color} its emails appear on its smartphone
        # {color:green}OK{color} the email with the invitation has a "created date" at 1970-01-01

        h4. Additional informations
        {code}
        4:15:19.313 [ERROR] {org.obm.push.handler.SyncHandler} [t40824/cSync/r38036/user@linagora.com] The field dtStamp is required
        java.lang.NullPointerException: The field dtStamp is required
        at com.google.common.base.Preconditions.checkNotNull(Preconditions.java:229) ~[guava-16.0.jar:na]
        at org.obm.push.bean.msmeetingrequest.MSMeetingRequest$Builder.build(MSMeetingRequest.java:185) ~[push-bean-3.1.0-rc3.jar:na]
        at org.obm.push.mail.ICalendarConverter.convertToMSMeetingRequest(ICalendarConverter.java:127) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MailViewToMSEmailConverterImpl.convertICalendar(MailViewToMSEmailConverterImpl.java:209) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MailViewToMSEmailConverterImpl.convert(MailViewToMSEmailConverterImpl.java:99) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MSEmailFetcher.fetch(MSEmailFetcher.java:87) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.EmailChangesFetcherImpl.fetchMSEmails(EmailChangesFetcherImpl.java:150) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.EmailChangesFetcherImpl.emailAdditions(EmailChangesFetcherImpl.java:104) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.EmailChangesFetcherImpl.fetch(EmailChangesFetcherImpl.java:77) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MailBackendImpl.fetchChanges(MailBackendImpl.java:291) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MailBackendImpl.continueWindowing(MailBackendImpl.java:283) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.push.mail.MailBackendImpl.getChanged(MailBackendImpl.java:256) ~[push-mail-3.1.0-rc3.jar:na]
        at org.obm.breakdownduration.BreakdownDurationInterceptor.invoke(BreakdownDurationInterceptor.java:58) ~[breakdown-duration-3.1.2.jar:na]
        at org.obm.push.ContentsExporter.getChanged(ContentsExporter.java:82) ~[push-core-3.1.0-rc3.jar:na]
        at org.obm.push.handler.SyncHandler.retreiveServerModifications(SyncHandler.java:288) ~[push-core-3.1.0-rc3.jar:na]
        at org.obm.push.handler.SyncHandler.doUpdates(SyncHandler.java:271) ~[push-core-3.1.0-rc3.jar:na]
        {code}
        Stephane COLSON made changes -
        Status Resolved [ 5 ] Stopped [ 10000 ]
        Stephane COLSON made changes -
        Status Stopped [ 10000 ] Closed [ 6 ]

          People

          • Assignee:
            Thomas HILAIRE
            Reporter:
            Thomas HILAIRE
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: