On Tue, Oct 01, 2024 at 12:29:15PM -0400, Tom Lane wrote: > Fujii Masao <masao.fu...@oss.nttdata.com> writes: >> Is a connection URL with whitespace, like >> "tcp:postgresql://localhost:5432/postgres?keepalives=1 & ...", >> considered valid? If not, the issue seems to be that ecpg adds unnecessary >> whitespace >> to the connection URL, especially after the "&" character. > > I agree with Sasaki-san that useKeepalives seems rather bogus: almost > every other place in fe-connect.c uses pqParseIntParam rather than > calling strtol directly, so why not this one?
Yes, it is a mistake to not use pqParseIntParam(), or parse_int_param() depending on the branch. This stuff has been introduced by 4f4061b2dde1, where I've spent some time making sure that leading and trailing whitespaces are discarded in this routine. See also these examples where whitespaces are OK in a connection URL: https://www.postgresql.org/message-id/20191021024020.GF1542%40paquier.xyz -- Michael
signature.asc
Description: PGP signature
