Uploaded image for project: 'Jira Data Center'
  1. Jira Data Center
  2. JRASERVER-61622

JIRA is not sending the Mention notification when using quotes

    XMLWordPrintable

Details

    Description

      NOTE: This bug report is for JIRA Server. Using JIRA Cloud? See the corresponding bug report.

      Summary

      JIRA is not sending the Mention notification when an user is mentioned in a comment if the Project's Notification Email has quotes.

      Steps to Reproduce

      1. Configure an Outgoing Mail in JIRA
      2. Create a new project;
      3. Edit the Project's Notification Email to have quotes on it:
      4. Create a ticket in this new project;
      5. Add a comment mentioning an user, to trigger the "user mentioned you" notification.

      Expected Results

      Receive the notification related to the mention.

      Actual Results

      The user is not receiving the notification related to the mention.
      The below exception is thrown in the atlassian-jira-outgoing.log file:

      2016-06-22 15:48:00,050 Sending mailitem To='demouser2@atlassian.com' Subject='Pedro Cora mentioned you on PX-1 (JIRA) ' From='"TESTING QUOTES" <atlassianjira@atlassian.com>' FromName='Pedro Cora JIRA' Cc='null' Bcc='null' ReplyTo='"TESTING QUOTES" <atlassianjira@atlassian.com>' InReplyTo='null' MimeType='text/html' Encoding='UTF-8' Multipart='javax.mail.internet.MimeMultipart@2773aa1' MessageId='null' ERROR anonymous    Mail Queue Service [c.a.mail.queue.MailQueueImpl] Error occurred in sending e-mail: To='demouser2@atlassian.com' Subject='Pedro Cora mentioned you on PX-1 (JIRA) ' From='"TESTING QUOTES" <atlassianjira@atlassian.com>' FromName='Pedro Cora JIRA' Cc='null' Bcc='null' ReplyTo='"TESTING QUOTES" <atlassianjira@atlassian.com>' InReplyTo='null' MimeType='text/html' Encoding='UTF-8' Multipart='javax.mail.internet.MimeMultipart@27f1553f' MessageId='null'
      com.atlassian.mail.MailException: javax.mail.internet.AddressException: Missing '"' in string ``"TESTING'' at position 8
      	at com.atlassian.mail.server.impl.SMTPMailServerImpl.sendWithMessageId(SMTPMailServerImpl.java:214)
      	at com.atlassian.mail.queue.SingleMailQueueItem.send(SingleMailQueueItem.java:44)
      	at com.atlassian.jira.mail.builder.RenderingMailQueueItem.send(RenderingMailQueueItem.java:33)
      	at com.atlassian.mail.queue.MailQueueImpl.sendBuffer(MailQueueImpl.java:66)
      	at com.atlassian.jira.mail.JiraMailQueue$1.apply(JiraMailQueue.java:51)
      	at com.atlassian.jira.mail.JiraMailQueue$1.apply(JiraMailQueue.java:48)
      	at com.atlassian.jira.util.velocity.DefaultVelocityRequestContextFactory.runWithStaticBaseUrl(DefaultVelocityRequestContextFactory.java:110)
      	at com.atlassian.jira.util.DefaultBaseUrl.runWithStaticBaseUrl(DefaultBaseUrl.java:50)
      	at com.atlassian.jira.mail.JiraMailQueue.sendBuffer(JiraMailQueue.java:48)
      	at com.atlassian.jira.service.services.mail.MailQueueService.run(MailQueueService.java:21)
      	at com.atlassian.jira.service.JiraServiceContainerImpl.run(JiraServiceContainerImpl.java:61)
      	at com.atlassian.jira.service.ServiceRunner.runService(ServiceRunner.java:62)
      	at com.atlassian.jira.service.ServiceRunner.runServiceId(ServiceRunner.java:44)
      	at com.atlassian.jira.service.ServiceRunner.runJob(ServiceRunner.java:32)
      	at com.atlassian.scheduler.core.JobLauncher.runJob(JobLauncher.java:153)
      	at com.atlassian.scheduler.core.JobLauncher.launchAndBuildResponse(JobLauncher.java:118)
      	at com.atlassian.scheduler.core.JobLauncher.launch(JobLauncher.java:97)
      	at com.atlassian.scheduler.caesium.impl.CaesiumSchedulerService.launchJob(CaesiumSchedulerService.java:401)
      	at com.atlassian.scheduler.caesium.impl.CaesiumSchedulerService.executeLocalJob(CaesiumSchedulerService.java:369)
      	at com.atlassian.scheduler.caesium.impl.CaesiumSchedulerService.executeQueuedJob(CaesiumSchedulerService.java:347)
      	at com.atlassian.scheduler.caesium.impl.CaesiumSchedulerService$1.consume(CaesiumSchedulerService.java:255)
      	at com.atlassian.scheduler.caesium.impl.CaesiumSchedulerService$1.consume(CaesiumSchedulerService.java:252)
      	at com.atlassian.scheduler.caesium.impl.SchedulerQueueWorker.executeJob(SchedulerQueueWorker.java:65)
      	at com.atlassian.scheduler.caesium.impl.SchedulerQueueWorker.executeNextJob(SchedulerQueueWorker.java:59)
      	at com.atlassian.scheduler.caesium.impl.SchedulerQueueWorker.run(SchedulerQueueWorker.java:34)
      	at java.lang.Thread.run(Thread.java:745)
      Caused by: javax.mail.internet.AddressException: Missing '"' in string ``"TESTING'' at position 8
      	at javax.mail.internet.InternetAddress.parse(InternetAddress.java:867)
      	at javax.mail.internet.InternetAddress.parse(InternetAddress.java:638)
      	at javax.mail.internet.InternetAddress.<init>(InternetAddress.java:111)
      	at com.atlassian.mail.MailUtils.parseAddresses(MailUtils.java:129)
      	at com.atlassian.mail.server.impl.util.MessageCreator.updateMimeMessage(MessageCreator.java:53)
      

      Notes

      More generally, the behaviour described occurs when using named emails with quotes surrounding the display-name which is an expected format according to the RFC5322.
      An example of such is "Name Surname" <name.surname@example.com. The https://www.rfc-editor.org/rfc/rfc5322#appendix-A.1.2 also provides an example as a valid format.

      If you add the quotes to the Outgoing Mail configuration and remove the email from the project's notification email (to use the default), the problem is not reproducible. It only occurs if you save an email with quotes directly in the project.

      Workaround

      Edit the Project's Notification email to something without quotes.

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              aquadrospetry Andre Quadros Petry (Inactive)
              Votes:
              11 Vote for this issue
              Watchers:
              13 Start watching this issue

              Dates

                Created:
                Updated: