Uploaded image for project: 'Confluence Data Center'
  1. Confluence Data Center
  2. CONFSERVER-6251

Error thown user email value is not set

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Medium
    • 2.2.2
    • 2.2
    • None

    Description

      When a page edit is committed and a user is set to watch that page, yet that user does not have a value stored in their user profile for email address, the following error is logged by Confluence.

      2006-05-25 15:38:25,683 ERROR [com.opensymphony.xwork.ActionSupport] doSaveEditPageBean An error occured while storing the requested page!
      java.lang.IllegalArgumentException: user cstivers has null email address for mail notification item with subject Dave Conn: Home (page edited)
      at com.atlassian.confluence.mail.template.MailNotificationQueueItem.checkUser(MailNotificationQueueItem.java:82)
      at com.atlassian.confluence.mail.template.MailNotificationQueueItem.<init>(MailNotificationQueueItem.java:56)
      at com.atlassian.confluence.mail.template.PreRenderedMailNotificationQueueItem.<init>(PreRenderedMailNotificationQueueItem.java:21)
      at com.atlassian.confluence.mail.template.PreRenderedMailNotificationQueueItem.createFromTemplateFileAndLocation(PreRenderedMailNotificationQueueItem.java:46)
      at com.atlassian.confluence.mail.template.PreRenderedMailNotificationQueueItem.createFromTemplateFileAndContext(PreRenderedMailNotificationQueueItem.java:99)
      at com.atlassian.confluence.mail.notification.listeners.AbstractNotificationsListener.sendNotification(AbstractNotificationsListener.java:44)
      at com.atlassian.confluence.mail.notification.listeners.AbstractNotificationsListener.sendNotifications(AbstractNotificationsListener.java:66)
      at com.atlassian.confluence.mail.notification.listeners.AbstractNotificationsListener.sendPageNotifications(AbstractNotificationsListener.java:51)
      ....etc....

      The problem is that the page is stored, but the page viewer will see an error message which states "An error has occured" without further information. The page has been stored correctly, but the editor will NOT know this. Subsequent edits will then throw an error stating that the editer is working with an outdated page (because the Confluence editor component will be using the version of the page BEFORE the error was thrown, yet the new page was already stored successfully.)
      This issue is EASY to work around for us, as we can simple define the "mail" attribute in our LDAP implementation for all users, however, Confluence should be able to gracefully store the page for the editor, while sending an error email to Confluence Administrators that the user WATCHING the page does not have an email address defined. If this action is too complex, Confluence should at LEAST have the ability to save the page WITHOUT error for the editor, while still logging that an error has occured.

      Attachments

        Activity

          People

            Unassigned Unassigned
            26635efb5c8b Clint Stivers
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: