Skip to content

Feat/kasmvnc webtop upgrade#1064

Open
fasiho7 wants to merge 3 commits intoOWASP:mainfrom
fasiho7:feat/kasmvnc-webtop-upgrade
Open

Feat/kasmvnc webtop upgrade#1064
fasiho7 wants to merge 3 commits intoOWASP:mainfrom
fasiho7:feat/kasmvnc-webtop-upgrade

Conversation

@fasiho7
Copy link

@fasiho7 fasiho7 commented Jan 23, 2026

Thank you for submitting a pull request to the WrongSecrets Party!

What kind of changes does this PR include?

  • Fixes or refactors
  • Platform support
  • A new feature
  • Additional documentation
  • Something else

Checklist:

  • All the contributions made are solely the work of me and my co-authors
  • I tested the changes in this PR (if applicable)
  • I added tests to ensure my change works (if applicable)
  • The PR passes pre-commit hooks and automated tests

- Fixes issue where Challenge 0 answer was missing from CSV export
- Supports both JSON (?format=json) and CSV (?format=csv) formats
- Includes all challenge answers (0-53) for CTFd integration
- Pin to latest KasmVNC webtop image (lscr.io/linuxserver/webtop:latest)
- Update container image references to use new webtop
- Change port from 3000/8080 to 6080 for KasmVNC compatibility
- Update proxy routing to use correct port for webtop access

Fixes compatibility with new KasmVNC-based webtop setup
{
name: 'virtualdesktop',
image: `jeroenwillemsen/wrongsecrets-desktop-k8s:${wrongSecretsDekstopTag}`,
image: `lscr.io/linuxserver/webtop:${wrongSecretsDekstopTag}`,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @fasiho7 ,
thank you for your PR!
I was wondering: have you tested this setup? does the desktop work then?
If so, we rather use the latest version of the jeroenwillemsen/wrongsecrets-desktop-k8s which also leverages a very new version of the lscr.io/linuxserver/webtop image.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants