Commons Email
  1. Commons Email
  2. EMAIL-70

Email.setMailSession() discards provided session if using authentication

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.1
    • Fix Version/s: 1.2
    • Labels:
      None
    • Environment:

      All

      Description

      The Email.setSession(Session aSession) method does NOT use the provided session value if the provided session is an authenticated session. In that case, a new Session instance is created and used instead. The newly created session uses a DefaultAuthenticator based on the username/password properties provided in the original session.

      This is a problem because the original session may have been created with a valid authenticator but without placing the password in the session properties. In this case, the newly constructed session will not work. Or, the original session may have been created with a custom authenticator, so again, the newly constructed session will not work.

      It seems to me that setSession() should simply set the provided session and not attempt to be smart about authentication. However, for backward compatibility, an improvement would be to offer a setRawSession() that simply sets this.session. Another improvement would be to also check for the presence of username and password properties before deciding to create a new session instance. If either of those properties is missing, use the provided session as-is.

        Activity

        Siegfried Goeschl made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Siegfried Goeschl made changes -
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Resolved [ 5 ]
        Siegfried Goeschl made changes -
        Fix Version/s 1.2 [ 12313573 ]
        Siegfried Goeschl made changes -
        Assignee Siegfried Goeschl [ sgoeschl ]
        Ben Speakmon made changes -
        Field Original Value New Value
        Affects Version/s 1.1 [ 12312230 ]
        F. Andy Seidl created issue -

          People

          • Assignee:
            Siegfried Goeschl
            Reporter:
            F. Andy Seidl
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development