Kevin Chung
Kevin Chung
Although likely it might just make more sense to just keep it reading as argument variables...
It looks at some point healthchecks changed to using a `--connection-info` switch. I think a better choice is to pass the information using envvars. https://github.com/CTFd/ctfcli/blob/d8e08daedd6d22159f5978e1619b9d7073aa07f8/ctfcli/cli/challenges.py#L913-L916
So I thought about this for awhile and I think this is possibly going in the wrong direction. Ideally everything needed to deploy a challenge should be in `challenge.yml`. I...
Shares sholud be configurable during setup and enabled by default
Closed by https://github.com/CTFd/CTFd/pull/2802
This could be useful. However I would prefer if the plugin raised specific exceptions that would be caught and presented to the user instead of catching any exception.
Perhaps we can also explore this idea for validation in flags which also have a custom programmable aspect behind it.
This is implemented in the ratelimit decorator. https://github.com/CTFd/CTFd/blob/master/CTFd/utils/decorators/__init__.py#L130
API rate limits should probably be defined in config and KPM can be defined in config with an override in the Admin Panel. A PR is okay.
I think decay doesn't make sense being in this PR. It also doesn't make sense as a bulk edit because decay doesn't exist on all challenges. Can you slim down...