Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1oe0PZ-0002us-S0 for pgsql-bugs@arkaria.postgresql.org; Thu, 29 Sep 2022 20:45:37 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.92) (envelope-from ) id 1oe0PY-0007Wj-OO for pgsql-bugs@arkaria.postgresql.org; Thu, 29 Sep 2022 20:45:36 +0000 Received: from makus.postgresql.org ([2001:4800:3e1:1::229]) by malur.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1oe0PY-0007Wa-G0 for pgsql-bugs@lists.postgresql.org; Thu, 29 Sep 2022 20:45:36 +0000 Received: from sss.pgh.pa.us ([66.207.139.130]) by makus.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1oe0PW-0005Rd-8c for pgsql-bugs@lists.postgresql.org; Thu, 29 Sep 2022 20:45:35 +0000 Received: from sss1.sss.pgh.pa.us (localhost [127.0.0.1]) by sss.pgh.pa.us (8.15.2/8.15.2) with ESMTP id 28TKjW5x395957; Thu, 29 Sep 2022 16:45:33 -0400 From: Tom Lane To: Jacob Champion cc: heath.lord@crunchydata.com, Daniel Gustafsson , pgsql-bugs@lists.postgresql.org Subject: Re: BUG #17625: In PG15 PQsslAttribute returns different values than PG14 when SSL is not in use for the connection In-reply-to: References: <17625-fc47c78b7d71b534@postgresql.org> <346263.1664480998@sss.pgh.pa.us> <360379.1664482591@sss.pgh.pa.us> Comments: In-reply-to Jacob Champion message dated "Thu, 29 Sep 2022 13:40:09 -0700" MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-ID: <395955.1664484332.1@sss.pgh.pa.us> Date: Thu, 29 Sep 2022 16:45:32 -0400 Message-ID: <395956.1664484332@sss.pgh.pa.us> List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk Jacob Champion writes: > On Thu, Sep 29, 2022 at 1:16 PM Tom Lane wrote: >> This'd still be broken for the >> multiple-libraries scenario, but I admit that that's pretty >> hypothetical. > Since the goal is to let clients decide which connection options to > hardcode based on the SSL implementation, I think it stays > forward-compatible with multiple libraries, as long as this API > returns the "default" library that you get if you're an older, > clueless client. We would need a new API of some sort to let newer > clients figure out their choices. Yeah, that was in my mind too: we could leave it like this as long as we define the result for (NULL, "library") as being the default SSL library choice. Good enough for now, then. I'll go tweak the documentation as per Daniel's thoughts and push. regards, tom lane