Skip to content

♻️ Migrate to DATABASE_URL instead of separate variables#2184

Draft
patrick91 wants to merge 1 commit intomasterfrom
migrate-to-database-url-instead-of-separate-variab
Draft

♻️ Migrate to DATABASE_URL instead of separate variables#2184
patrick91 wants to merge 1 commit intomasterfrom
migrate-to-database-url-instead-of-separate-variab

Conversation

@patrick91
Copy link
Contributor

@patrick91 patrick91 commented Feb 13, 2026

No description provided.

@at146
Copy link

at146 commented Feb 13, 2026

I don't know why you want to

@atmosmaciel
Copy link

I understand that this is an important modification for abstracting (configuring) database connections, regardless of the engine chosen by the user.

@patrick91
Copy link
Contributor Author

@at146 sorry for the late reply, I was actually reconsidering this a bit

I usually migrate to a single env var for this, just for convenience (also most cloud's integrations expose that env variable, so it is easier to deploy)

I understand that this is an important modification for abstracting (configuring) database connections, regardless of the engine chosen by the user.

you mean being able to do: DATABASE_URL=postgres+engine://host/db?

@atmosmaciel
Copy link

@at146 sorry for the late reply, I was actually reconsidering this a bit

I usually migrate to a single env var for this, just for convenience (also most cloud's integrations expose that env variable, so it is easier to deploy)

I understand that this is an important modification for abstracting (configuring) database connections, regardless of the engine chosen by the user.

you mean being able to do: DATABASE_URL=postgres+engine://host/db?

Exactly. Using DATABASE_URL is more of a semantic and practical convenience than something necessarily more technical. It removes the need for a SQLALCHEMY_DATABASE_URI environment and multiple lines defining POSTGRES_*.

I don't know about all the impacts involved, but apparently it wouldn't be a big problem. It would be interesting if a maintainer could clarify some doubts about this.

@at146 any specific reason why this modification wouldn't be useful?

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants