Skip to content

Commit

Permalink
Fixed "REPLACE_ENV_VARIABLES_EXCLUDE_PATHS" breaking cfg replacer whe…
Browse files Browse the repository at this point in the history
…n set. (#598)
  • Loading branch information
Trikolon authored Jul 26, 2020
1 parent 82b8401 commit a486458
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 7 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -329,7 +329,7 @@ Specific files can be excluded by listing their name (without path) in the varia
Paths can be excluded by listing them in the variable `REPLACE_ENV_VARIABLES_EXCLUDE_PATHS`. Path
excludes are recursive. Here is an example:
```
REPLACE_ENV_VARIABLES_EXCLUDE_PATHS="/data/plugins/Essentials/userdata/ /data/plugins/MyPlugin/"
REPLACE_ENV_VARIABLES_EXCLUDE_PATHS="/data/plugins/Essentials/userdata /data/plugins/MyPlugin"
```

Here is a full example where we want to replace values inside a `database.yml`.
Expand Down
15 changes: 9 additions & 6 deletions start-finalSetupEnvVariables
Original file line number Diff line number Diff line change
Expand Up @@ -6,20 +6,21 @@

if isTrue "${REPLACE_ENV_VARIABLES}"; then
log "Replacing env variables in configs that match the prefix $ENV_VARIABLE_PREFIX..."
findExcludes=

# File excludes
fileExcludes=
for f in ${REPLACE_ENV_VARIABLES_EXCLUDES}; do
findExcludes="${findExcludes} -not -name $f"
fileExcludes="${fileExcludes} -not -name $f"
done

# Directory excludes (recursive)
dirExcludes=$(join_by " -o -path " ${REPLACE_ENV_VARIABLES_EXCLUDE_PATHS})
if [[ $dirExcludes ]]; then
findExcludes+=" -type d ( -path ${dirExcludes} ) -prune"
dirExcludes=" -type d ( -path ${dirExcludes} ) -prune -o"
fi

isDebugging && echo "Using find exclusions: $findExcludes"
isDebugging && echo "Using find file exclusions: $fileExcludes"
isDebugging && echo "Using find directory exclusions: $dirExcludes"

while IFS='=' read -r name value ; do
# check if name of env variable matches the prefix
Expand All @@ -34,10 +35,12 @@ if isTrue "${REPLACE_ENV_VARIABLES}"; then
fi

log "Replacing $name with $value ..."
find /data/ -type f \
find /data/ \
$dirExcludes \
-type f \
\( -name "*.yml" -or -name "*.yaml" -or -name "*.txt" -or -name "*.cfg" \
-or -name "*.conf" -or -name "*.properties" \) \
$findExcludes \
$fileExcludes \
-exec sed -i 's#${'"$name"'}#'"$value"'#g' {} \;
fi
done < <(env)
Expand Down

0 comments on commit a486458

Please sign in to comment.