Skip to content

Authentication and Password Resolution Improvements#62

Merged
ric-v merged 3 commits intodev-asterix:mainfrom
ric-v:main
Mar 15, 2026
Merged

Authentication and Password Resolution Improvements#62
ric-v merged 3 commits intodev-asterix:mainfrom
ric-v:main

Conversation

@ric-v
Copy link
Member

@ric-v ric-v commented Mar 15, 2026

This pull request introduces several improvements and fixes focused on authentication and connection handling, especially regarding .pgpass file support and password resolution logic. The changes enhance compatibility with standard PostgreSQL password file conventions, improve fallback reliability, and make the authentication process more robust and transparent.

Authentication and Password Resolution Improvements:

  • Added native, backwards-compatible support for .pgpass files (and Windows %APPDATA%\postgresql\pgpass.conf), including explicit parsing and resolution logic to match standard PostgreSQL behavior. [1] [2] [3]
  • Improved password resolution order: now prefers SecretStorage, then inline settings, and finally .pgpass (only for explicit usernames), with clear fallback to handle trust authentication and SCRAM errors gracefully. [1] [2]
  • Enhanced .pgpass lookup scope to avoid implicit machine name environments, ensuring backward compatibility for local trust authentications. [1] [2]

Connection and Client Configuration Reliability:

  • Fixed an issue where DatabaseTreeProvider could strip configuration details (such as inline passwords and sslmode) during fallback calculations by ensuring the full connection object is passed through. [1] [2]
  • Improved logging and error messaging for password resolution, making it easier to diagnose authentication issues and understand which password source was used.

Other:

  • Updated version to 0.8.7 and added a detailed changelog entry for these enhancements and fixes. [1] [2]

ric-v added 3 commits March 15, 2026 23:57
…n logic, fixing authentication resilience and connection configuration stripping.
@ric-v ric-v merged commit 5740d5c into dev-asterix:main Mar 15, 2026
3 of 13 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.

1 participant