• Icon: Improvement Improvement
    • Resolution: Done
    • Icon: Minor Minor
    • mailer-plugin
    • - Jenkins version: 2.152
      - Mailer plugin version: 1.22
      - JCasC plugin version: 1.3
      - OS: Linux
    • mailer-1.25

      There is no possibility to set mailer credentials "as code". Jenkins Configuration as Code handles smtpHost, smtpPort and useSsl fields, but it doesn't handle useSMTPAuth, smtpAuthUserName and smtpAuthPasswordSecret fields.

      Expected result

      JCasC plugin should handle useSMTPAuth, smtpAuthUserName and smtpAuthPasswordSecret fields.

      Example configuration
      unclassified:
        mailer:
          adminAddress: admin <admin@dummy.tld>
          replyToAddress: admin <admin@dummy.tld>
          charset: UTF-8
      
          smtpHost: smtp.mailgun.org
          smtpPort: 587
          useSsl: false
      
          useSMTPAuth: true
          smtpAuthUserName: XXXXX@XXXXX.mailgun.org
          smtpAuthPasswordSecret: XXXXX

          [JENKINS-55109] Unable to set credentials with CasC plugin

          Damian Łysiak created issue -

          Many thanks in advance for somebody involved in this improvement  

          Damian Łysiak added a comment - Many thanks in advance for somebody involved in this improvement  
          Damian Łysiak made changes -
          Description Original: There is no possibility to set mailer credentials "as code". Jenkins Configuration as Code handles {{smtpHost}}, {{smtpPort}} and {{useSsl}} fields, but it doesn't handle {{useSMTPAuth}}, {{smtpAuthUserName}} and {{smtpAuthPasswordSecret}} fields.
          h4. Expected result

          JCasC plugin should handle {{useSMTPAuth}}, {{smtpAuthUserName}} and {{smtpAuthPasswordSecret}} fields.
          h4. Example configuration
          {code:java}unclassified:
            mailer:
              adminAddress: admin <admin@dummy.tld>
              replyToAddress: admin <admin@dummy.tld>
              charset: UTF-8

              smtpHost: smtp.mailgun.org
              smtpPort: 587
              useSsl: false

              useSMTPAuth: true
              smtpAuthUserName: XXXXX@XXXXX.mailgun.org
              smtpAuthPasswordSecret: XXXXX{code}
          New: There is no possibility to set mailer credentials "as code". Jenkins Configuration as Code handles {{smtpHost}}, {{smtpPort}} and {{useSsl}} fields, but it doesn't handle {{useSMTPAuth}}, {{smtpAuthUserName}} and {{smtpAuthPasswordSecret}} fields.
          h5. Expected result

          JCasC plugin should handle {{useSMTPAuth}}, {{smtpAuthUserName}} and {{smtpAuthPasswordSecret}} fields.
          h5. Example configuration
          {code:java}
          unclassified:
            mailer:
              adminAddress: admin <admin@dummy.tld>
              replyToAddress: admin <admin@dummy.tld>
              charset: UTF-8

              smtpHost: smtp.mailgun.org
              smtpPort: 587
              useSsl: false

              useSMTPAuth: true
              smtpAuthUserName: XXXXX@XXXXX.mailgun.org
              smtpAuthPasswordSecret: XXXXX{code}
          Oleg Nenashev made changes -
          Labels New: jcasc-compatibility

          I'm not sure mailer is compatible with CasC until https://github.com/jenkinsci/mailer-plugin/pull/39 is integrated. 

          I'm in the process of its review. 

          Adrien Lecharpentier added a comment - I'm not sure mailer is compatible with CasC until https://github.com/jenkinsci/mailer-plugin/pull/39  is integrated.  I'm in the process of its review. 
          René Scheibe made changes -
          Remote Link New: This issue links to "PR#39 (Web Link)" [ 22441 ]
          Francisco Fernández made changes -
          Assignee Original: Adrien Lecharpentier [ alecharp ] New: Francisco Fernández [ fcojfernandez ]

          Discussed with alecharp: I will try to land  https://github.com/jenkinsci/mailer-plugin/pull/39 in a new PR adding some tests for JCasC 

          Francisco Fernández added a comment - Discussed with alecharp : I will try to land   https://github.com/jenkinsci/mailer-plugin/pull/39  in a new PR adding some tests for JCasC 
          Francisco Fernández made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Francisco Fernández made changes -
          Status Original: In Progress [ 3 ] New: In Review [ 10005 ]

            fcojfernandez Francisco Fernández
            damlys Damian Łysiak
            Votes:
            1 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated:
              Resolved: