Skip to content

Support generic PostgreSQL DB args to allow for ssl-mode settings#215

Open
darthhexx wants to merge 4 commits into
danny-avila:mainfrom
darthhexx:feat/postgresql-database-argument-support
Open

Support generic PostgreSQL DB args to allow for ssl-mode settings#215
darthhexx wants to merge 4 commits into
danny-avila:mainfrom
darthhexx:feat/postgresql-database-argument-support

Conversation

@darthhexx

@darthhexx darthhexx commented Oct 20, 2025

Copy link
Copy Markdown

Adds support for generic connection string arguments for PostgreSQL so that SSL mode and TLS certificates can be configured.

For example:

POSTGRES_DATABASE_ARGS="sslmode=verify-ca&sslrootcert=/app/certs/server-ca.pem&sslcert=/app/certs/client-cert.pem&sslkey=/app/certs/client-key.pem"

@darthhexx darthhexx changed the title Support generic PostgreSQL DB args for ssl-mode settings Support generic PostgreSQL DB args to allow for ssl-mode settings Oct 20, 2025
@danny-avila danny-avila requested a review from Copilot December 16, 2025 21:47

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds support for configuring generic PostgreSQL connection string arguments via the POSTGRES_DATABASE_ARGS environment variable, enabling SSL mode and TLS certificate configuration for PostgreSQL connections.

Key Changes:

  • Introduces POSTGRES_DATABASE_ARGS environment variable to accept custom connection parameters
  • Appends these arguments as query parameters to the PostgreSQL connection string for TCP connections

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread app/config.py
Comment thread app/config.py
Comment thread app/config.py
darthhexx and others added 3 commits January 27, 2026 12:07
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.

2 participants