dockerExecuteOnKubernetes - hide sensitive information (#733)

* dockerExecuteOnKubernetes - hide yaml by default
* hide step parameters to not leak sensitive parameter values into the log
* add more details to log output

test/groovy/HandlePipelineStepErrorsTest.groovy

@@ -79,7 +79,7 @@ class HandlePipelineStepErrorsTest extends BasePiperTest {
             // asserts
             assertThat(isReported, is(true))
             assertThat(loggingRule.log, containsString('--- An error occurred in the library step: testStep'))
-            assertThat(loggingRule.log, containsString('[something:anything]'))
+            assertThat(loggingRule.log, containsString('to show step parameters, set verbose:true'))
         }
     }
     

vars/dockerExecuteOnKubernetes.groovy

@@ -14,7 +14,12 @@ import hudson.AbortException
 @Field def PLUGIN_ID_KUBERNETES = 'kubernetes'
 
 @Field Set GENERAL_CONFIG_KEYS = [
-    'jenkinsKubernetes'
+    'jenkinsKubernetes',
+    /**
+     * Print more detailed information into the log.
+     * @possibleValues `true`, `false`
+     */
+    'verbose'
 ]
 @Field Set STEP_CONFIG_KEYS = GENERAL_CONFIG_KEYS.plus([
     /**
@@ -152,7 +157,9 @@ def getOptions(config) {
     if (config.nodeSelector) {
         options.nodeSelector = config.nodeSelector
     }
-
+    if (!config.verbose) {
+        options.showRawYaml = false
+    }
     return options
 }
 

vars/handlePipelineStepErrors.groovy

@@ -119,7 +119,7 @@ private String formatErrorMessage(Map config, error){
         libraryDocumentationUrl: config.libraryDocumentationUrl,
         libraryRepositoryUrl: config.libraryRepositoryUrl,
         stepName: config.stepName,
-        stepParameters: config.stepParameters?.toString()
+        stepParameters: (config.stepParameters?.verbose == true) ? config.stepParameters?.toString() : '*** to show step parameters, set verbose:true in general pipeline configuration\n*** WARNING: this may reveal sensitive information. ***'
     ]
     return SimpleTemplateEngine
         .newInstance()