Skip to content

A vault secret to read arbitrary variables from and pass to children #46

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Southclaws opened this issue Mar 20, 2020 · 2 comments
Closed

Comments

@Southclaws
Copy link
Collaborator

This can also be used to solve #24

Essentially, any variables prefixed with PICO_ can be reserved for Pico use, such as PICO_GIT_USERNAME to solve #24

Then, any other variables can just be passed to every future task.

The secret can sit at VAULT_CONFIG_PATH which will default to pico. Along with the default base path, this would place the default config path at /secret/pico which seems logical.

@Southclaws
Copy link
Collaborator Author

Currently, all environment variables from the process are passed to children. This is insecure and can leak information about the host machine into children and reduce the boundary between applications.

This should be disabled and only variables that are prefixed should be passed through (with the prefix stripped).

And the same logic applied to secret stores.

@Southclaws
Copy link
Collaborator Author

The PR for #24 landed but with the names GIT_* instead of PICO_ prefixed.

Southclaws added a commit that referenced this issue Mar 23, 2020
Adds a new option: pass-env, which when true will pass the pico process environment to children. Defaults to false to promote separation of environments.

Adds support for passing prefixed variables from the global Pico. The prefix is GLOBAL_ and is not configurable because I felt the config flags are growing.

Adds some better unit tests for execution config and environment merging.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant