Display connection info and SSL status on startup - #815
Conversation
REV Code Review Report
BLOCKING ISSUES (0)No blocking issues found. POTENTIAL ISSUES (1)LOW
Summary
Notes:
SOC2 COMPLIANCE (2)HIGH SOC2: Code Review — No assigned reviewer
HIGH SOC2: Linked Issue — No issue reference in description
Overall: PASSED — Implementation is clean and correct. Confirm CI passes before merging. REV-assisted review (AI analysis) |
| // Server version (full, including distro build info when present) | ||
| println!("Server: PostgreSQL {server_ver}"); | ||
| // Connection details + SSL status — matching psql startup output. | ||
| println!("{}", connection::connection_info(&resolved)); |
Print "You are connected to database..." and SSL connection details on initial interactive connection, matching psql's startup output. Also add a trailing blank line after the help hint, like psql does. Closes #789 https://claude.ai/code/session_01NzCjbwZ5vrLXyLV442BRhq
Point users to ~/.config/rpg/config.toml in both the "not configured" and "key not set" cases so they know where to fix it. https://claude.ai/code/session_01NzCjbwZ5vrLXyLV442BRhq
9d90446 to
2813af2
Compare
|
Superseded by #817, which includes all changes from this PR (connection info display, config.toml hint, blank line) plus the CodeQL security fix that required using |
Summary
Enhanced the startup output to display connection details and SSL status information, matching the output format of psql.
Key Changes
connection::connection_info()functionImplementation Details
The connection details are now printed immediately after the PostgreSQL server version information, providing users with visibility into their connection parameters and SSL status at startup. This aligns the application's output with standard psql behavior, making the startup experience more familiar to PostgreSQL users.
https://claude.ai/code/session_01NzCjbwZ5vrLXyLV442BRhq