Uploaded image for project: 'James Server'
  1. James Server
  2. JAMES-434

NPE on empty sender and AUTHenticated user

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2.0
    • Fix Version/s: 2.3.0
    • Component/s: SMTPServer
    • Labels:
      None

      Description

      If an AUTHenticated user try to send a message with an empty sender james throws an NullPointerException.

      I've created a test for this but I don't know what the best behaviour is:
      should we simply return a 503 5.7.1 Incorrect Authentication for Specified Email Address ?
      or should we allow empty senders for AUTHenticated users?

        Activity

        Hide
        norman Norman Maurer added a comment -

        Whats the status of that ? It seems to be fixed.. I can not get this NullPointer .

        Show
        norman Norman Maurer added a comment - Whats the status of that ? It seems to be fixed.. I can not get this NullPointer .
        Hide
        bago Stefano Bagnara added a comment -

        Marking this as fixed. Return a 503 error if a null sender is used.
        If we think that it is better to handle null senders from authenticated networks we should open a new issue.

        Show
        bago Stefano Bagnara added a comment - Marking this as fixed. Return a 503 error if a null sender is used. If we think that it is better to handle null senders from authenticated networks we should open a new issue.
        Hide
        danny@apache.org Danny Angus added a comment -

        Closing issue fixed in released version.

        Show
        danny@apache.org Danny Angus added a comment - Closing issue fixed in released version.

          People

          • Assignee:
            bago Stefano Bagnara
            Reporter:
            bago Stefano Bagnara
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development