Commons Email
  1. Commons Email
  2. EMAIL-48

[email] API for TLS authentication and JNDI mail session integration


    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.0
    • Fix Version/s: None
    • Labels:
    • Environment:

      Operating System: other
      Platform: Other


      I'd like to suggest two improvements for facilities in API :

      1. TLS support
      Add methods to use TLS during authentification when needed.
      => this require adding methods such "void setTLS(boolean
      withTLS)" and "boolean isTLS()" and adding the property
      mail.smtp.starttls.enable=true to MailSessionObject
      =>in version 1.0, TLS can be supported by adding the property
      "mail.smtp.starttls.enable" to system before sending message.
      It woudd be more friendly to just have a setTLS method,
      moreover this property woud just be added to MaillSession
      object and not to system properties.

      2. JNDI support
      In J2EE container, mail session can be obtained from JNDI
      direcory. A method like "void setMailSessionFromJNDI(String
      jndi name)" would ne more friendly too.
      => in version 1.0 we have to do the JNDI search in code. This
      method would allow to simply pass the JNDI name of mail sesssion.


        Q. Werty created issue -
        Henri Yandell made changes -
        Field Original Value New Value
        issue.field.bugzillaimportkey 37782 12342775
        Henri Yandell made changes -
        Project Commons [ 12310458 ] Commons Email [ 12310474 ]
        Affects Version/s 1.0 Final [ 12311651 ]
        Key COM-2623 EMAIL-48
        Assignee Jakarta Commons Developers Mailing List [ ]
        Component/s Email [ 12311114 ]
        Henri Yandell made changes -
        Affects Version/s 1.0 Final [ 12311731 ]
        Siegfried Goeschl made changes -
        Status Resolved [ 5 ] Closed [ 6 ]


          • Assignee:
            Q. Werty
          • Votes:
            0 Vote for this issue
            0 Start watching this issue


            • Created: