Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cmgr should validate picoctf required params #2

Open
dbrumley opened this issue Nov 25, 2024 · 1 comment
Open

cmgr should validate picoctf required params #2

dbrumley opened this issue Nov 25, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@dbrumley
Copy link

The author attribute and the point config needs to be present in problems for the picoctf platform, but cmgr raises no warnings when not present.

@dbrumley dbrumley added the enhancement New feature or request label Nov 30, 2024
@dmartin
Copy link

dmartin commented Dec 2, 2024

In theory cmgr could be used by multiple consuming clients, not just the picoCTF platform, so I have avoided hardcoding anything pico-specific so far.
But as we are probably the only actual existing client application, I agree these would be fine to add (at least as nonfatal warnings).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants