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

Add an option to hide PasswordParameters in build logs

      EnvInject plugin already depends on PasswordParameter in several places in order to hide its values. It would be great to use the plugin in order to hide password parameters in build logs as well.

          [JENKINS-24130] Add an option to hide PasswordParameters in build logs

          Oleg Nenashev added a comment -

          Oleg Nenashev added a comment - https://github.com/jenkinsci/envinject-plugin/pull/31

          Code changed in jenkins
          User: Oleg Nenashev
          Path:
          src/main/java/org/jenkinsci/plugins/envinject/EnvInjectPasswordWrapper.java
          src/main/resources/org/jenkinsci/plugins/envinject/EnvInjectPasswordWrapper/config.jelly
          src/main/webapp/help-buildWrapperMaskPasswordParameters.html
          http://jenkins-ci.org/commit/envinject-plugin/dcf57030c5e1d72db09dc02934c3c5db3c2c6f4d
          Log:
          [FIXED JENKINS-24130] - Added an option to mask PasswordParameters

          This is a straightforward implementation, an additional re-factoring would be useful.
          The option is enabled by default, but this behavior can be changed in order to prevent any compatibility issues.

          Signed-off-by: Oleg Nenashev <o.v.nenashev@gmail.com>

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oleg Nenashev Path: src/main/java/org/jenkinsci/plugins/envinject/EnvInjectPasswordWrapper.java src/main/resources/org/jenkinsci/plugins/envinject/EnvInjectPasswordWrapper/config.jelly src/main/webapp/help-buildWrapperMaskPasswordParameters.html http://jenkins-ci.org/commit/envinject-plugin/dcf57030c5e1d72db09dc02934c3c5db3c2c6f4d Log: [FIXED JENKINS-24130] - Added an option to mask PasswordParameters This is a straightforward implementation, an additional re-factoring would be useful. The option is enabled by default, but this behavior can be changed in order to prevent any compatibility issues. Signed-off-by: Oleg Nenashev <o.v.nenashev@gmail.com>

          Code changed in jenkins
          User: Gregory Boissinot
          Path:
          src/main/java/org/jenkinsci/plugins/envinject/EnvInjectPasswordWrapper.java
          src/main/resources/org/jenkinsci/plugins/envinject/EnvInjectPasswordWrapper/config.jelly
          src/main/webapp/help-buildWrapperMaskPasswordParameters.html
          http://jenkins-ci.org/commit/envinject-plugin/20d8390f44f44034c46af8e29e1a7ed8e3b9fbb4
          Log:
          Merge pull request #31 from synopsys-arc-oss/JENKINS-24130

          [FIXED JENKINS-24130] - Added an option to mask PasswordParameters

          Compare: https://github.com/jenkinsci/envinject-plugin/compare/13d590235f56...20d8390f44f4

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Gregory Boissinot Path: src/main/java/org/jenkinsci/plugins/envinject/EnvInjectPasswordWrapper.java src/main/resources/org/jenkinsci/plugins/envinject/EnvInjectPasswordWrapper/config.jelly src/main/webapp/help-buildWrapperMaskPasswordParameters.html http://jenkins-ci.org/commit/envinject-plugin/20d8390f44f44034c46af8e29e1a7ed8e3b9fbb4 Log: Merge pull request #31 from synopsys-arc-oss/ JENKINS-24130 [FIXED JENKINS-24130] - Added an option to mask PasswordParameters Compare: https://github.com/jenkinsci/envinject-plugin/compare/13d590235f56...20d8390f44f4

            oleg_nenashev Oleg Nenashev
            oleg_nenashev Oleg Nenashev
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: