Skip to content

Conversation

@jakubgs
Copy link
Member

@jakubgs jakubgs commented Nov 7, 2025

Recent upgrade of Jenkins and Git Paramter plugin caused these to appear:

[2025-11-07T12:07:08.487Z] Scheduling project: status-desktop » e2e » prs
Invalid parameter value: (StringParameterValue) GIT_REF='6e42ff2a5cb10933332d6722e1c19babc9973c07'

Which are caused by recently patched vulnerability in the plugin:

We could disable this check with:

gitParameterDefinition.allowAnyParameterValue=true

But just using a String paramater is simpler and more convenient.

@jakubgs jakubgs requested a review from caybro November 7, 2025 13:00
@jakubgs jakubgs self-assigned this Nov 7, 2025
@jakubgs jakubgs requested a review from a team as a code owner November 7, 2025 13:00
@status-im-auto
Copy link
Member

status-im-auto commented Nov 7, 2025

Jenkins Builds

Click to see older builds (19)
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 91972cc #1 2025-11-07 13:09:04 ~8 min tests/nim 📄log
✔️ 91972cc #1 2025-11-07 13:09:43 ~8 min android/arm64 🤖apk 📲
91972cc #1 2025-11-07 13:11:37 ~10 min macos/aarch64-nwaku 📄log
✔️ 91972cc #1 2025-11-07 13:14:56 ~13 min tests/ui 📄log
✔️ 91972cc #1 2025-11-07 13:15:40 ~14 min ios/aarch64 📱ipa
✔️ 91972cc #1 2025-11-07 13:16:12 ~15 min macos/aarch64 🍎dmg
✖️ 91972cc #1 2025-11-07 13:18:48 ~17 min linux/x86_64 📦tgz
✔️ 91972cc #1 2025-11-07 13:23:38 ~22 min linux/x86_64-nwaku 📦tgz
✖️ 91972cc #1 2025-11-07 13:34:58 ~33 min windows/x86_64 💿exe
91972cc #2 2025-11-07 13:39:18 ~5 min macos/aarch64-nwaku 📄log
✖️ 91972cc #2 2025-11-07 13:49:34 ~15 min linux/x86_64 📦tgz
7cc7ade #3 2025-11-07 14:04:00 ~5 min macos/aarch64-nwaku 📄log
✔️ 7cc7ade #2 2025-11-07 14:06:21 ~8 min tests/nim 📄log
✔️ 7cc7ade #2 2025-11-07 14:08:01 ~10 min android/arm64 🤖apk 📲
✔️ 7cc7ade #2 2025-11-07 14:09:43 ~11 min macos/aarch64 🍎dmg
✔️ 7cc7ade #2 2025-11-07 14:10:58 ~12 min ios/aarch64 📱ipa
✔️ 7cc7ade #2 2025-11-07 14:12:51 ~14 min tests/ui 📄log
✔️ 7cc7ade #2 2025-11-07 14:13:55 ~15 min linux/x86_64-nwaku 📦tgz
✖️ 7cc7ade #3 2025-11-07 14:15:50 ~17 min linux/x86_64 📦tgz
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 30ac3c2 #3 2025-11-07 14:37:01 ~12 min android/arm64 🤖apk 📲
✔️ 30ac3c2 #3 2025-11-07 14:38:12 ~13 min tests/nim 📄log
30ac3c2 #3 2025-11-07 14:38:19 ~13 min macos/aarch64 📄log
✔️ 30ac3c2 #3 2025-11-07 14:42:45 ~18 min ios/aarch64 📱ipa
✖️ 30ac3c2 #4 2025-11-07 14:45:32 ~21 min linux/x86_64 📦tgz
✔️ 30ac3c2 #4 2025-11-07 14:45:39 ~20 min macos/aarch64-nwaku 🍎dmg
✔️ 30ac3c2 #3 2025-11-07 14:45:44 ~21 min tests/ui 📄log
✔️ 30ac3c2 #3 2025-11-07 14:51:51 ~27 min linux/x86_64-nwaku 📦tgz
✖️ 30ac3c2 #3 2025-11-07 15:01:26 ~36 min windows/x86_64 💿exe
✔️ 5fbadab #4 2025-11-07 15:46:29 ~6 min tests/nim 📄log
✔️ 5fbadab #4 2025-11-07 15:50:11 ~10 min android/arm64 🤖apk 📲
✔️ 5fbadab #4 2025-11-07 15:52:30 ~12 min macos/aarch64 🍎dmg
✔️ 5fbadab #4 2025-11-07 15:53:55 ~13 min tests/ui 📄log
✔️ 5fbadab #4 2025-11-07 15:54:03 ~13 min ios/aarch64 📱ipa
✔️ 5fbadab #5 2025-11-07 15:56:31 ~16 min linux/x86_64 📦tgz
✔️ 5fbadab #4 2025-11-07 16:02:22 ~22 min linux/x86_64-nwaku 📦tgz
✔️ 5fbadab #5 2025-11-07 16:03:25 ~23 min macos/aarch64-nwaku 🍎dmg
✖️ 5fbadab #4 2025-11-07 16:13:31 ~33 min windows/x86_64 💿exe
✔️ 5fbadab pr19234 2025-11-07 16:20:46 ~24 min tests/e2e 📊rpt
✖️ 5fbadab #5 2025-11-07 17:36:59 ~36 min windows/x86_64 💿exe

@jakubgs jakubgs force-pushed the ci/drop-git-paramater branch 2 times, most recently from 7cc7ade to 30ac3c2 Compare November 7, 2025 14:24
Recent upgrade of Jenkins and Git Paramter plugin caused these to appear:

[2025-11-07T12:07:08.487Z] Scheduling project: status-desktop » e2e » prs
Invalid parameter value: (StringParameterValue) GIT_REF='6e42ff2a5cb10933332d6722e1c19babc9973c07'

Which are caused by recently patched vulnerability in the plugin:
https://www.jenkins.io/security/advisory/2025-07-09/#SECURITY-3419
https://github.com/jenkinsci/git-parameter-plugin/releases/tag/444.vca_b_84d3703c2

We could disable this check with:

gitParameterDefinition.allowAnyParameterValue=true

But just using a String paramater is simpler and more convenient.

Signed-off-by: Jakub Sokołowski <jakub@status.im>
@jakubgs jakubgs force-pushed the ci/drop-git-paramater branch from 30ac3c2 to 5fbadab Compare November 7, 2025 15:39
@jakubgs jakubgs merged commit 5fbadab into master Nov 7, 2025
11 of 12 checks passed
@jakubgs jakubgs deleted the ci/drop-git-paramater branch November 7, 2025 17:50
@jakubgs
Copy link
Member Author

jakubgs commented Nov 7, 2025

Keep in mind feature branches will have to rebase to take effect.

@friofry
Copy link
Contributor

friofry commented Nov 8, 2025

Keep in mind feature branches will have to rebase to take effect.

I've rebased by branch, but the error is the same https://ci.status.im/blue/organizations/jenkins/status-desktop%2Fprs%2Fwindows%2Fx86_64%2Fpackage/detail/PR-19116/7/pipeline

[2025-11-07T21:55:11.125Z] Scheduling project: status-desktop » e2e » prs-windows
Invalid parameter value: (StringParameterValue) GIT_REF='c3b1b11695708ecaffd014b9bb2304d62f5e2f5d'

git log:

* c3b1b1169 (origin/ab/issue-18545-connector-backend, ab/issue-18545-connector-backend) feat(dapp-browser-backend)_: async calls
* f0979777d (origin/master, origin/HEAD) docs(dev_setup): update mobile readme to clarify how to install
* 5fbadab66 fix(ci): replace gitParameter with string parameter

@friofry
Copy link
Contributor

friofry commented Nov 8, 2025

Using env.BRANCH_NAME (as in the Jenkinsfile) helped:

948ccd7

@jakubgs
Copy link
Member Author

jakubgs commented Nov 8, 2025

That change doesn't make sense. We want to test an exact commit, not the branch. But it works as a temp fix to let it run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants