comic / grand-challenge.org

A platform for end-to-end development of machine learning solutions in biomedical imaging
https://grand-challenge.org
Apache License 2.0
168 stars 50 forks source link

Move `DockerExecutor` to tests #3273

Closed jmsmkn closed 3 months ago

jmsmkn commented 3 months ago

This PR moves the DockerExecutor to the tests app so that it is not present in the production containers and renames it to InsecureDockerExecutor. It also adds a deployment check to make sure users are not using this in production, and raises ImproperlyConfigured if DEBUG=False and pytest is not being run. If DEBUG=True we add a critical error message on every initialisation. This should prevent users from using it in production.