From f42b2938368bcd8a41e85b010f634cd80fcb3746 Mon Sep 17 00:00:00 2001 From: Jose Diaz-Gonzalez Date: Sun, 13 Dec 2020 02:10:49 -0500 Subject: [PATCH] feat: remove reference to 'whitelist' The term doesn't even 'make sense' in this context. 'allowed_prefixes' is a much better variable name. --- plugins/scheduler-docker-local/scheduler-inspect | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/plugins/scheduler-docker-local/scheduler-inspect b/plugins/scheduler-docker-local/scheduler-inspect index efe42b23d..d269615bc 100755 --- a/plugins/scheduler-docker-local/scheduler-inspect +++ b/plugins/scheduler-docker-local/scheduler-inspect @@ -23,11 +23,11 @@ output = [] data = json.load(sys.stdin) for container in data: env = [] - whitelist = ['CACHE_PATH=', 'DOCKER_', 'DOKKU_', 'DYNO=', 'PATH=', 'PORT=', 'USER='] + allowed_prefixes = ['CACHE_PATH=', 'DOCKER_', 'DOKKU_', 'DYNO=', 'PATH=', 'PORT=', 'USER='] for e in container['Config']['Env']: append = False - for w in whitelist: - if e.startswith(w): + for s in allowed_prefixes: + if e.startswith(s): append = True break if append: