Skip to content

Support latest rCTF and k8s version - #17

Merged
DarinMao merged 3 commits into
TJCSec:masterfrom
trixter-osec:fix/latest-rctf
Oct 30, 2025
Merged

Support latest rCTF and k8s version#17
DarinMao merged 3 commits into
TJCSec:masterfrom
trixter-osec:fix/latest-rctf

Conversation

@trixter-osec

Copy link
Copy Markdown
Contributor

This PR updates klodd to support:

  • latest rCTF version (redpwn version is now maintained by osec here)
  • updates the auth check to match goodUserSelfData which was changed part of a rewrite that is now merged (otter-sec/rctf@c0a086b)
  • latest k8s version seems to close the watch() connection after 5 minutes, causing ECONNRESET that eventually causes the entire klodd process to crash, this solution is probably not the cleanest way to solve this, but updating the dependencies requires significant effort due to having to upgrade Node.js version + some of the other dependencies due to being pinned to only specific Node.js versions
  • building from source does not seem to work for me due to one of the dependencies requiring Node >=18, so the Node.js version has been bumped respectively without having to upgrade rest of the dependencies

@DarinMao

Copy link
Copy Markdown
Member

The Kubernetes change was patched in during an event a few months ago and seems to work well. Someone was supposed to PR this but everyone forgot about it 🙃

To be honest, I'm not sure what the right way to do it is. The original thinking was that if there were problems then Klodd could just be quickly restarted, but in practice it seems to take some time. Hopefully this is a good enough solution.

DarinMao
DarinMao previously approved these changes Oct 30, 2025
Comment thread packages/server/src/app/rctf.js Outdated
@DarinMao
DarinMao merged commit 5be8ff5 into TJCSec:master Oct 30, 2025
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