pgsql: Do not hardcode PG_PROTOCOL_LATEST in NegotiateProtocolVersion - Mailing list pgsql-committers

From Robert Haas
Subject pgsql: Do not hardcode PG_PROTOCOL_LATEST in NegotiateProtocolVersion
Date
Msg-id E1sebjC-004IJo-3Y@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Do not hardcode PG_PROTOCOL_LATEST in NegotiateProtocolVersion

We shouldn't ask the client to use a protocol version later than the
one that they requested. To avoid that, if the client requests a
version newer than the latest one we support, set FrontendProtocol
to the latest version we support, not the requested version. Then,
use that value when building the NegotiateProtocolVersion message.
(It seems good on general principle to avoid setting FrontendProtocol
to a version we don't support, anyway.)

None of this really matters right now, because we only support a
single protocol version, but if that ever changes, we'll need this.

Jelte Fennema-Nio, reviewed by me and incorporating some of my
proposed wording

Discussion: https://postgr.es/m/CAGECzQTyXDNtMXdq2L-Wp=OvOCPa07r6+U_MGb==h90MrfT+fQ@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/516b87502dc1f99adb5126aa70cc796d53648c92

Modified Files
--------------
src/backend/tcop/backend_startup.c | 19 +++++++++++++------
1 file changed, 13 insertions(+), 6 deletions(-)


pgsql-committers by date:

Previous
From: Dean Rasheed
Date:
Subject: pgsql: Optimise numeric multiplication using base-NBASE^2 arithmetic.
Next
From: Tom Lane
Date:
Subject: pgsql: Clean up indentation and whitespace inconsistencies in ecpg.