Re: ProcessStartupPacket(): database_name and user_name truncation

Bertrand Drouvot <bertranddrouvot.pg@gmail.com>

From: "Drouvot, Bertrand" <bertranddrouvot.pg@gmail.com>
To: Michael Paquier <michael@paquier.xyz>, Nathan Bossart <nathandbossart@gmail.com>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, Kyotaro Horiguchi <horikyota.ntt@gmail.com>, pgsql-hackers@lists.postgresql.org
Date: 2023-06-22T06:10:30Z
Lists: pgsql-hackers
Hi,

On 6/22/23 1:37 AM, Michael Paquier wrote:
> On Wed, Jun 21, 2023 at 12:55:15PM -0700, Nathan Bossart wrote:
>> LGTM.  I think this can wait for v17 since the current behavior has been
>> around since 2001 and AFAIK this is the first report.  While it's arguably
>> a bug fix, the patch also breaks some cases that work today.
> 
> Agreed that anything discussed on this thread does not warrant a
> backpatch.

Fully agree, the CF entry [1] has been tagged as "Target Version 17".

[1] https://commitfest.postgresql.org/43/4383/

Regards,

-- 
Bertrand Drouvot
PostgreSQL Contributors Team
RDS Open Source Databases
Amazon Web Services: https://aws.amazon.com



Commits

  1. Don't truncate database and user names in startup packets.