Skip to content

[DB] Use Postgres startup encoding without SET NAMES - #1

Merged
kageiit merged 3 commits into
gitar-0.6.11from
fix/postgres-startup-encoding
Oct 7, 2026
Merged

kageiit merged 3 commits into
gitar-0.6.11from
fix/postgres-startup-encoding

Conversation

@kageiit

@kageiit kageiit commented Oct 7, 2026 •

Copy link
Copy Markdown

Postgres encoding uses the startup packet.
The patched tokio-postgres driver already requests UTF8 when connecting. Quaint repeated it with SET NAMES on every connection, causing all 9,946 RDS Proxy warnings in the production window to report session pinning. Remove the redundant SQL and retain explicit schema initialization.

Validation: local PostgreSQL 17 tests pass for UTF-8 round trips and custom search paths. Formatting passes. All-target clippy completes with the existing approximate-constant test lint allowed, retaining unrelated warnings.

Gitar needs a separate consumer version-pin PR. Existing sessions remain pinned until they reconnect after deployment.

Fork pull requests now run the supported Postgres compatibility workflow, including the encoding and search-path regressions against PostgreSQL 17. The upstream main branch retains its original multi-connector workflow.

Decisions made while unattended
  • Scoped fork CI to its supported Postgres contract as the upstream matrix fails deprecated actions and unrelated warning baselines
  • Added the encoding regressions beside the existing TLS checks so the narrower workflow still exercises this change

@kageiit
kageiit force-pushed the fix/postgres-startup-encoding branch from 9f6f9c3 to 622c886 Compare October 7, 2026 09:04
@kageiit
kageiit merged commit 8f88674 into gitar-0.6.11 Oct 7, 2026
2 checks passed
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