this#3491 doesn't fix it ?

this#3491 doesn't fix it ?

No, on line#906 it's actually returning immediately if the version >= 9.0 so it never reaches that code. Also, the postgres logging doen't include the application name in the initial connection logs if it is not included at startup.

OK, I think we need to add it back. I think the runInitialQueries needs to be tightened up and we need to document this better.
runInitialQueries should still setextra_float_digits=3for versions 9-12 andextra_float_digits=2` for versions less than 9

Thanks!

I believe the latest changes will run the initial queries only when necessary.
On PG 12 and later no initial queries are required assuming the assumMinServerVersion is set correctly.
On PG 9-11.x only the extra digits query is required if assumeMinServerVersion is set correctly.

I was able to add a unit test that fails for the original code, when assumeMinServerVersion was set to >= 9.0 resulting in the application name being blank. However, I don't see a straight-forward way to verify that it's being set in the startup parameters rather than via the initial queries (in the test harness at least). If you have any ideas, I would appreciate it.

I also added a note about this use-case into the documentation, not sure if it belongs anywhere else.