-
Notifications
You must be signed in to change notification settings - Fork 68
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
Ability to set CTFd config #89
Comments
I have interest in this specific issue and may be able to help. Would you accept a PR on this? If so, do you have any recommendations regarding potential implementations? |
@SmylerMC yes I would accept a PR for this. I don't think there's much trick to it but I am concerned that in the Ideally I think that we would want this to be something like:
Perhaps that means that we rename the top level config to something else. |
What about using YAML for the CTFd config file? I feel like that would make things more coherent, ctfcli related files being in TOML and CTFd related files in YAML. Regarding the commands, I like the idea of changing |
Another solution I was tinkering with for the commands would be to keep Example sub-commands could be:
|
I am working on this now. I like the idea of the instance subcommand. The issue I've run into now is that the .ctf/config structure is pretty bad because it is ini based and the server config stores a variety of things. |
Well by the structure of the Configs model both key and value should be strings or null so I suppose we can work with that. |
We need a way to share CTFd configuration between events. It's too easy to forget that a configuration setting needs to be set.
The text was updated successfully, but these errors were encountered: