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

Documentation needed around environment variables with lists #1801

Closed
fmmoret opened this issue Oct 2, 2021 · 1 comment · Fixed by ory/docs#527
Closed

Documentation needed around environment variables with lists #1801

fmmoret opened this issue Oct 2, 2021 · 1 comment · Fixed by ory/docs#527

Comments

@fmmoret
Copy link

fmmoret commented Oct 2, 2021

Is your feature request related to a problem? Please describe.

The secrets rotation doc (https://www.ory.sh/kratos/docs/guides/secret-key-rotation/#route-identifier) and the config reference (https://www.ory.sh/kratos/docs/reference/configuration) don't tell you what kind of delimiter is used for all env variables that take lists: SECRETS_DEFAULT, SECRETS_COOKIE, SELFSERVICE_WHITELISTED_RETURN_URLS

Describe the solution you'd like

I'd like to know how to supply environment variable list values and I want to avoid generating secrets/values that would contain the delimiter.

Describe alternatives you've considered

I tried figuring out how config parsing & translation works in kratos. The dependency chain was too long for me to keep track / follow.

P.S. I would love to know the answer here in this thread before that documentation makes it live if anyone knows the answer 🙏

@aeneasr
Copy link
Member

aeneasr commented Oct 2, 2021

You can provide them in CSV format or JSON format!

aeneasr added a commit to ory/docs that referenced this issue Dec 30, 2021
aeneasr added a commit that referenced this issue Dec 30, 2021
aeneasr added a commit to ory/docs that referenced this issue Dec 30, 2021
aeneasr added a commit that referenced this issue Dec 30, 2021
peturgeorgievv pushed a commit to senteca/kratos-fork that referenced this issue Jun 30, 2023
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

Successfully merging a pull request may close this issue.

2 participants