Skip to content

Commit 43ef7ca

Browse files
mparamas-zzclaude
andcommitted
ci: fix Secret.trim() crash by making TARGET_NODE_PASSWORD a string param
The password() parameter type made params.TARGET_NODE_PASSWORD a hudson.util.Secret, which has no .trim() method — so the Copy Artifacts stage crashed with MissingMethodException before doing any work, and all downstream stages were skipped. A Secret also interpolates to its encrypted form in "SSHPASS=${...}", which would have broken sshpass auth even past the crash. Switch it to a string param so both the validation and the SSHPASS env interpolation receive plaintext. Note: string params are not masked in the UI; use a Jenkins credential if masking is required. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent 38e4fe6 commit 43ef7ca

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

Jenkinsfile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,10 +52,10 @@ properties([
5252
defaultValue: '',
5353
description: '(required when TARGET_NODE_IP is set) SSH username on the flashing host.'
5454
),
55-
password(
55+
string(
5656
name: 'TARGET_NODE_PASSWORD',
5757
defaultValue: '',
58-
description: '(required when TARGET_NODE_IP is set) SSH password on the flashing host. Passed via the SSHPASS env var, never on the command line.'
58+
description: '(required when TARGET_NODE_IP is set) SSH password on the flashing host. Passed via the SSHPASS env var, never on the command line. NOTE: a string param is not masked in the UI; use a Jenkins credential if masking is required.'
5959
),
6060
string(
6161
name: 'TARGET_DEST_DIR',

0 commit comments

Comments
 (0)