Re: Andres Freund
> > > The fprintf suggests that since 5dc1e42b4 PQescapeIdentifier ignores its len.
> >
> > Ugh, yes. Need something like the attached.
>
> I just pushed this fix, together with an expansion of test_escape.c. With the
> expanded test both uses of strlen() are detected.
FTR, this is also caught by pygresql's regression tests:
test_inserttable_with_dotted_table_name
(tests.test_classic_connection.TestInserttable.test_inserttable_with_dotted_table_name)... ERROR
https://ci.debian.net/packages/p/pygresql/testing/amd64/57838998/https://qa.debian.org/excuses.php?package=postgresql-17
What's missing in the PG regression tests to see that problem?
Christoph
