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

feat: add support for Postgres database provider #79

Merged
merged 6 commits into from
Dec 12, 2024

Conversation

stonith404
Copy link
Owner

@stonith404 stonith404 commented Dec 12, 2024

With this PR it is now possible to use Postgres instead of Sqlite as the database provider.

The following environment variables were added that allows the admin to configure the database provider:

Variable Default Value Description
DB_PROVIDER sqlite The database provider you want to use. Currently sqlite and postgres are supported.
SQLITE_DB_PATH data/pocket-id.db The path to the SQLite database. This gets ignored if you didn't set DB_PROVIDER to sqlite.
POSTGRES_CONNECTION_STRING - The connection string to your Postgres database. This gets ignored if you didn't set DB_PROVIDER to postgres. A connection string can look like this: postgresql://user:password@host:5432/pocket-id.

This PR can be tested with the stonith404/pocket-id:postgres-development Docker image.

@stonith404 stonith404 marked this pull request as ready for review December 12, 2024 16:21
@stonith404 stonith404 merged commit 9d20a98 into main Dec 12, 2024
6 checks passed
@stonith404 stonith404 deleted the feat/postgres-support branch December 12, 2024 16:21
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 this pull request may close these issues.

1 participant