Uploaded image for project: 'Continuum'
  1. Continuum
  2. CONTINUUM-1674

mail authentication with username/password doesn't work

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Open
    • Major
    • Resolution: Unresolved
    • 1.1, 1.4.1
    • None
    • Notifier - Mail
    • Debian 4.0, Tomcat 6.0.14, jdk 1.6.0 u3
    • Patch

    Description

      I use continuum as tomcat webapp.

      If you use a mailserver with authentication, you can define following in the context.xml of your webapp/META-INF:

      <Resource name="mail/Session"
      auth="Container"
      type="javax.mail.Session"
      mail.smtp.auth="true"
      mail.smtp.host="mailserver"
      mail.smtp.port="25"
      mail.smtp.user="username"
      password="password"/>

      The problem is, that the parameter password will be ignored. I found the problem in:

      org.codehaus.plexus.mailsender.javamail.JndiJavamailMailSender.getSession

      ->

      Context ctx = new InitialContext();
      Session s = (Session)ctx.lookup(jndiSessionName);

      The session doesn't contain a property called "password".

      My dirty workaround to solve the problem:

      renamed password to mail.smtp.password (in context.xml) and wrote a class which extends JndiJavamailMailSender with overriden getSession:

      public Session getSession() throws MailSenderException

      { Session sess = super.getSession(); Properties prop = sess.getProperties(); prop.setProperty(JndiJavamailMailSender.MAIL_SMTP_PASSWORD, prop.getProperty("mail.smtp.password")); return Session.getInstance(prop, null); }

      Maybe you set the value of the constant MAIL_SMTP_PASSWORD to "mail.stmp.password" and everything is fine!

      Attachments

        1. plex-javamail-2.patch
          0.8 kB
          Victor S.
        2. plex-javamail.patch
          0.5 kB
          Victor S.

        Issue Links

          Activity

            People

              Unassigned Unassigned
              jrx Jan Rieger
              Votes:
              1 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated: