Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-23630

Update to new environment variable APIs

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      PasswordParameterValue needs to be updated. buildEnvVars with Map is deprecated; should use the EnvVars overload. Should not upcase the variable name (or should also set the original name). Should override isSensitive. MaskPasswordsBuildWrapper should also override makeSensitiveBuildVariables.

        Attachments

          Issue Links

            Activity

            Hide
            jglick Jesse Glick added a comment -

            Until JENKINS-23447 is fixed, marking variables as sensitive is pointless when EnvInject is also installed.

            Show
            jglick Jesse Glick added a comment - Until JENKINS-23447 is fixed, marking variables as sensitive is pointless when EnvInject is also installed.
            Hide
            oleg_nenashev Oleg Nenashev added a comment -

            Taking the issue

            Show
            oleg_nenashev Oleg Nenashev added a comment - Taking the issue
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Oleg Nenashev
            Path:
            src/main/java/com/michelin/cio/hudson/plugins/maskpasswords/MaskPasswordsBuildWrapper.java
            src/main/java/com/michelin/cio/hudson/plugins/passwordparam/PasswordParameterValue.java
            http://jenkins-ci.org/commit/mask-passwords-plugin/e12c6cb9f157419bc529c15daad9f49668aea6ab
            Log:
            JENKINS-23630 - Honor the sensitive variables engine in Mask Psswords

            This change implements API for sensitive variables, which is required to hide passwords in plugins like EnvInject.

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oleg Nenashev Path: src/main/java/com/michelin/cio/hudson/plugins/maskpasswords/MaskPasswordsBuildWrapper.java src/main/java/com/michelin/cio/hudson/plugins/passwordparam/PasswordParameterValue.java http://jenkins-ci.org/commit/mask-passwords-plugin/e12c6cb9f157419bc529c15daad9f49668aea6ab Log: JENKINS-23630 - Honor the sensitive variables engine in Mask Psswords This change implements API for sensitive variables, which is required to hide passwords in plugins like EnvInject.
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Oleg Nenashev
            Path:
            src/main/java/com/michelin/cio/hudson/plugins/maskpasswords/MaskPasswordsBuildWrapper.java
            src/main/java/com/michelin/cio/hudson/plugins/passwordparam/PasswordParameterValue.java
            http://jenkins-ci.org/commit/mask-passwords-plugin/a591ed97cfd9465bec7d9daa13f26c5644825307
            Log:
            JENKINS-23630 - Honor the sensitive variables engine in Mask Psswords

            This change implements API for sensitive variables, which is required to hide passwords in plugins like EnvInject.

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oleg Nenashev Path: src/main/java/com/michelin/cio/hudson/plugins/maskpasswords/MaskPasswordsBuildWrapper.java src/main/java/com/michelin/cio/hudson/plugins/passwordparam/PasswordParameterValue.java http://jenkins-ci.org/commit/mask-passwords-plugin/a591ed97cfd9465bec7d9daa13f26c5644825307 Log: JENKINS-23630 - Honor the sensitive variables engine in Mask Psswords This change implements API for sensitive variables, which is required to hide passwords in plugins like EnvInject.
            Hide
            oleg_nenashev Oleg Nenashev added a comment -

            JENKINS-25821 will be closed in my PR instead

            Show
            oleg_nenashev Oleg Nenashev added a comment - JENKINS-25821 will be closed in my PR instead
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Oleg Nenashev
            Path:
            src/main/java/com/michelin/cio/hudson/plugins/maskpasswords/MaskPasswordsBuildWrapper.java
            src/main/java/com/michelin/cio/hudson/plugins/passwordparam/PasswordParameterValue.java
            http://jenkins-ci.org/commit/mask-passwords-plugin/48bedb99ae0db1cc2f18df438bc9cb86ea7ea695
            Log:
            Merge pull request #4 from jenkinsci/JENKINS-23630-sensitive-vars

            [FIXED JENKINS-25821] - Honor the sensitive variables engine in Mask Passwords

            Compare: https://github.com/jenkinsci/mask-passwords-plugin/compare/08ca0e0efcf5...48bedb99ae0d

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oleg Nenashev Path: src/main/java/com/michelin/cio/hudson/plugins/maskpasswords/MaskPasswordsBuildWrapper.java src/main/java/com/michelin/cio/hudson/plugins/passwordparam/PasswordParameterValue.java http://jenkins-ci.org/commit/mask-passwords-plugin/48bedb99ae0db1cc2f18df438bc9cb86ea7ea695 Log: Merge pull request #4 from jenkinsci/ JENKINS-23630 -sensitive-vars [FIXED JENKINS-25821] - Honor the sensitive variables engine in Mask Passwords Compare: https://github.com/jenkinsci/mask-passwords-plugin/compare/08ca0e0efcf5...48bedb99ae0d
            Hide
            oleg_nenashev Oleg Nenashev added a comment -

            It has been fixed in 2.7.4 long-long ago

            Show
            oleg_nenashev Oleg Nenashev added a comment - It has been fixed in 2.7.4 long-long ago

              People

              Assignee:
              oleg_nenashev Oleg Nenashev
              Reporter:
              jglick Jesse Glick
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: