Re: Psql meta-command conninfo+

From: Hunaid Sohail <hunaidpgml(at)gmail(dot)com>
To: pgsql-hackers(at)postgresql(dot)org
Cc: Maiquel Grassi <grassi(at)hotmail(dot)com(dot)br>, Jim Jones <jim(dot)jones(at)uni-muenster(dot)de>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, Nathan Bossart <nathandbossart(at)gmail(dot)com>, "Imseih (AWS), Sami" <simseih(at)amazon(dot)com>, Peter Eisentraut <peter(at)eisentraut(dot)org>, Pavel Luzanov <p(dot)luzanov(at)postgrespro(dot)ru>, Erik Wienhold <ewie(at)ewie(dot)name>, "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com>
Subject: Re: Psql meta-command conninfo+
Date: 2024-10-30 06:07:27
Message-ID: CAMWA6yZ0vSDzjMjqYMyKDzQaa474BJ6XLhNQ3ApuCST5d+j1nw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

I have attached a new patch that incorporates the approach suggested by
David. The documentation has also been updated.

$ bin/psql "port=5430 sslmode=disable dbname=postgres" -x -h localhost
psql (18devel)
Type "help" for help.

postgres=# \conninfo+
Connection Information
-[ RECORD 1 ]+----------
Database | postgres
Client User | hunaid
Host | localhost
Host Address | 127.0.0.1
Port | 5430
Options |

Current Status
-[ RECORD 1 ]--------+------
Protocol Version | 3
Password Used | false
GSSAPI Authenticated | false
Backend PID | 26268

Server Parameter Settings
-[ RECORD 1 ]---------+-------
Superuser | true
Client Encoding | UTF8
Server Encoding | UTF8
Session Authorization | hunaid

Connection Encryption
-[ RECORD 1 ]--+------
SSL Connection | false

$ bin/psql "port=5430 sslmode=require dbname=postgres" -x -h localhost
psql (18devel)
SSL connection (protocol: TLSv1.3, cipher: TLS_AES_256_GCM_SHA384,
compression: off, ALPN: postgresql)
Type "help" for help.

postgres=# \conninfo+ E
Connection Encryption
-[ RECORD 1 ]--+-----------------------
SSL Connection | true
Library | OpenSSL
Protocol | TLSv1.3
Key Bits | 256
Cipher | TLS_AES_256_GCM_SHA384
Compression | off
ALPN | postgresql

I’m unsure if we need to expand the documentation further. I would
appreciate your suggestions on this.

Regards,
Hunaid Sohail

On Mon, Oct 7, 2024 at 9:31 PM David G. Johnston <david(dot)g(dot)johnston(at)gmail(dot)com>
wrote:

> On Sun, Oct 6, 2024 at 11:17 PM Hunaid Sohail <hunaidpgml(at)gmail(dot)com>
> wrote:
>
>>
>> PQpass - no need
>>
>
> I would include this as presence/absence.
>
> I concur on all of the rest.
>
>
>>
>> For PQparameterStatus, some parameters are already used.
>> server_version and application_name were already discussed and removed in
>> v12 and v29 respectively. Do we need other parameters?
>>
>
> Ok, I'll need to go read the reasoning for why they are deemed unneeded
> and form an opinion one way or the other.
>
>
>>
>>> Within that framework having \conninfo[+[CSE][…]] be the command -
>>> printing out only the table specified would be the behavior (specifying no
>>> suffix letters prints all three) - would be an option.
>>>
>>
>> 3 separate tables without suffix?
>>
>
> Yes, the tables need headers specific to their categories.
>
> I do like the idea of having 4 though, placing settings into their own.
> Premised on having all or most of the available parameters being on the
> table. If it only ends up being a few of them then keeping those in
> the status table makes sense.
>
> David J.
>
>>

Attachment Content-Type Size
v36-0001-Add-psql-meta-command-conninfo.patch application/x-patch 16.2 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Tatsuo Ishii 2024-10-30 06:49:19 Re: protocol-level wait-for-LSN
Previous Message jian he 2024-10-30 03:57:35 Re: Wrong result when enable_partitionwise_join is on if collation of PartitionKey and Column is different.