From: Tom Lane <tgl@sss.pgh.pa.us>
To: Andres Freund <andres@anarazel.de>
cc: Andrew Dunstan <andrew@dunslane.net>, Noah Misch <noah@leadboat.com>,
        pgsql-hackers@postgresql.org, pgsql@j-davis.com
Subject: Re: Decision by Monday: PQescapeString() vs. encoding violation
In-reply-to: <7vmfuavitlhjgruidll67y5cd4s3m66m4x7c4yxezkxlo4dxvv@4haeja2xdn2q>
References: <20250215012712.45@rfd.leadboat.com>
 <f9e447e5-d94f-45f0-8ccb-2570cd4419c9@dunslane.net>
 <77190.1739642883@sss.pgh.pa.us>
 <7vmfuavitlhjgruidll67y5cd4s3m66m4x7c4yxezkxlo4dxvv@4haeja2xdn2q>
Comments: In-reply-to Andres Freund <andres@anarazel.de>
	message dated "Sat, 15 Feb 2025 13:23:51 -0500"
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-ID: <80373.1739644510.1@sss.pgh.pa.us>
Content-Transfer-Encoding: quoted-printable
Date: Sat, 15 Feb 2025 13:35:10 -0500
Message-ID: <80374.1739644510@sss.pgh.pa.us>
Archived-At: 
 <https://www.postgresql.org/message-id/80374.1739644510%40sss.pgh.pa.us>
Precedence: bulk

Andres Freund <andres@anarazel.de> writes:
> It seems that nobody is arguing against the "just skip one byte" behavio=
ur, so
> I'm inclined to push this fairly soon, even if Noah's "24 hours" haven't=
 quite
> elapsed.  A few more cycles in the buildfarm wouldn't hurt.

Agreed.  I thought there would be more discussion, but it seems
nobody really objects to changing this.

The other thing that was discussed in the security thread was
modifying PQescapeStringInternal and PQescapeInternal to produce
no more than one complaint about invalid multibyte characters,
on the grounds that input that's just plain in some other encoding
would otherwise produce a ton of repetitive messages.  That seems
trivial enough to mechanize with a bool already_complained flag,
so I think we should incorporate that refinement while we're here.
I can write that patch if you're busy.

			regards, tom lane