Skip to content
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

Use StepAbs for all files in the ca.json #372

Open
maraino opened this issue Sep 18, 2020 · 3 comments · May be fixed by #1894
Open

Use StepAbs for all files in the ca.json #372

maraino opened this issue Sep 18, 2020 · 3 comments · May be fixed by #1894

Comments

@maraino
Copy link
Contributor

maraino commented Sep 18, 2020

Description

Currently, some files in the ca.json, like the certificate templates, can be configured relative to the STEPPATH, for example, if a template file is set in the ca.json to "templates/x509/leaf.tpl" the location of the file would be relative to the STEPPATH, it will be $(step path)/templates/x509/leaf.tpl.

We should make this functionality available across all the files in the ca.json. That way, we can have things like:

{
   "root": "certs/root_ca.crt",
   "crt": "certs/intermediate_ca.crt",
   "key": "secrets/intermeidate_ca_key",
   "...": {}
}
@maraino maraino added enhancement needs triage Waiting for discussion / prioritization by team labels Sep 18, 2020
@maraino maraino changed the title Use StepAbs for all files in the ca.json Use StepAbs for all files in the ca.json Sep 18, 2020
@maraino
Copy link
Contributor Author

maraino commented Sep 18, 2020

@dopey dopey removed the needs triage Waiting for discussion / prioritization by team label Sep 23, 2020
@tashian
Copy link
Contributor

tashian commented Feb 22, 2021

+1 if it is a quick fix, as it would be helpful for NixOS integration

@marten-seemann
Copy link

The function was renamed to Abs, and is now located in a different repo: https://github.com/smallstep/cli-utils/blob/655922af79575526cb99469944535c5d267ed24a/step/config.go#L166-L192

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants