diff --git a/cli/run.go b/cli/run.go index 0dc320d7b..13dd8aad0 100644 --- a/cli/run.go +++ b/cli/run.go @@ -33,7 +33,7 @@ func Run(version, commit string) { PersistentPreRun: func(cmd *cobra.Command, args []string) { dirs := []map[string]os.FileMode{ {config.ABRA_DIR: 0764}, - {config.SERVERS_DIR: 0600}, + {config.SERVERS_DIR: 0700}, {config.RECIPES_DIR: 0764}, {config.LOGS_DIR: 0764}, } diff --git a/tests/integration/dirs.bats b/tests/integration/dirs.bats index c391e0dca..b6d4acc0f 100644 --- a/tests/integration/dirs.bats +++ b/tests/integration/dirs.bats @@ -39,5 +39,5 @@ setup(){ assert_not_exists "$ABRA_DIR/catalogue" server_dir_perms=$(stat -c "%a" "$ABRA_DIR/servers") - assert_equal $server_dir_perms "600" + assert_equal $server_dir_perms "700" } diff --git a/tests/integration/server_add.bats b/tests/integration/server_add.bats index 6de57779c..d31d064ba 100644 --- a/tests/integration/server_add.bats +++ b/tests/integration/server_add.bats @@ -25,6 +25,9 @@ teardown(){ assert_output --partial "$TEST_SERVER" assert bash -c "docker context ls | grep -q $TEST_SERVER" + + server_dir_perms=$(stat -c "%a" "$ABRA_DIR/servers/$TEST_SERVER") + assert_equal $server_dir_perms "600" } @test "error if using name and --local together" { @@ -39,6 +42,9 @@ teardown(){ assert_exists "$ABRA_DIR/servers/default" assert bash -c "docker context ls | grep -q default" assert_output --partial 'local server successfully added' + + server_dir_perms=$(stat -c "%a" "$ABRA_DIR/servers/$TEST_SERVER") + assert_equal $server_dir_perms "600" } @test "create local server fails when no docker swarm" {