Skip to content

Commit

Permalink
feat: remove reference to 'whitelist'
Browse files Browse the repository at this point in the history
The term doesn't even 'make sense' in this context. 'allowed_prefixes' is a much better variable name.
  • Loading branch information
josegonzalez committed Dec 13, 2020
1 parent ca2de6a commit f42b293
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions plugins/scheduler-docker-local/scheduler-inspect
Original file line number Diff line number Diff line change
Expand Up @@ -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:
Expand Down

0 comments on commit f42b293

Please sign in to comment.