Merge pull request #76 from fzerorubigd/master

pass password to redis-cli instance on connect sub-command
This commit is contained in:
Jose Diaz-Gonzalez
2017-04-22 13:12:38 -06:00
committed by GitHub
3 changed files with 12 additions and 5 deletions

View File

@@ -104,10 +104,17 @@ service_start() {
fi
}
service_password() {
local SERVICE="$1"
local SERVICE_ROOT="$PLUGIN_DATA_ROOT/$SERVICE"
local PASSWORD="$(cat "$SERVICE_ROOT/PASSWORD")"
echo "$PASSWORD"
}
service_url() {
local SERVICE="$1"
local SERVICE_ROOT="$PLUGIN_DATA_ROOT/$SERVICE"
local PASSWORD="$(cat "$SERVICE_ROOT/PASSWORD")"
local PASSWORD="$(service_password "$SERVICE")"
local SERVICE_ALIAS="$(service_alias "$SERVICE")"
echo "$PLUGIN_SCHEME://$SERVICE:$PASSWORD@$SERVICE_ALIAS:${PLUGIN_DATASTORE_PORTS[0]}"
}

View File

@@ -13,9 +13,10 @@ redis-connect-cmd() {
verify_service_name "$SERVICE"
SERVICE_ROOT="$PLUGIN_DATA_ROOT/$SERVICE"
SERVICE_NAME="$(get_service_name "$SERVICE")"
PASSWORD="$(service_password "$SERVICE")"
has_tty && SERVICE_TTY_OPTS="-t"
docker exec -i $SERVICE_TTY_OPTS "$SERVICE_NAME" redis-cli
docker exec -i $SERVICE_TTY_OPTS "$SERVICE_NAME" redis-cli -a "$PASSWORD"
}
redis-connect-cmd "$@"

View File

@@ -24,6 +24,5 @@ teardown() {
@test "($PLUGIN_COMMAND_PREFIX:connect) success" {
export ECHO_DOCKER_COMMAND="true"
run dokku "$PLUGIN_COMMAND_PREFIX:connect" l
assert_output 'docker exec -i -t dokku.redis.l redis-cli'
assert_contains 'docker exec -i -t dokku.redis.l redis-cli -a'
}