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

Allow usage of POSTGRESQL_CONNECTION_URI instead of separate db/password/user in helm chart #28

Open
SkeLLLa opened this issue Nov 22, 2023 · 4 comments · Fixed by #29

Comments

@SkeLLLa
Copy link
Contributor

SkeLLLa commented Nov 22, 2023

Allowing usage of POSTGRESQL_CONNECTION_URI which simplify setups and sometimes it works much better, since through connection URI various other parameters could be configured, while currently through standard db/pass/user is not possible.

@SkeLLLa SkeLLLa changed the title Allow usage of POSTGRESQL_CONNECTION_URI instead of separate db/password/user Allow usage of POSTGRESQL_CONNECTION_URI instead of separate db/password/user in helm chart Nov 22, 2023
@rishabhpoddar
Copy link
Member

Hey @SkeLLLa we already have this config. See here.

Closing this issue for now, but feel free to reopen.

@SkeLLLa
Copy link
Contributor Author

SkeLLLa commented Nov 23, 2023

@rishabhpoddar there's no such option in helm config.

The link you've provided leading to completely different product - supertokens-postgresql-plugin and I'm talking about helm chart.

@rishabhpoddar
Copy link
Member

Right. So the helm chart is a community contributed chart. The docker image itself (supertokens-docker-postgresql) is built using the repo link i had shared, so the docker env var does accept this config.

Please feel free to create a PR for the helm chart to add this env var.

@SkeLLLa
Copy link
Contributor Author

SkeLLLa commented Nov 25, 2023

@rishabhpoddar done.

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