TypeError: Cannot read properties of undefined (reading 'replace')

[alexgleason]

I'm getting some kind of error, but the error handler itself has an error, so I'm not sure what the error is. 😃

error: Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'replace')
      stack: { value: err.stack + query.origin.replace(/.*\n/, '\n'), enumerable: options.debug },
                                               ^
    at queryError (https://deno.land/x/postgresjs@v3.4.4/src/connection.js:392:48)
    at errored (https://deno.land/x/postgresjs@v3.4.4/src/connection.js:387:17)
    at error (https://deno.land/x/postgresjs@v3.4.4/src/connection.js:379:5)
    at https://deno.land/x/postgresjs@v3.4.4/polyfills.js:138:30
    at Array.forEach (<anonymous>)
    at call (https://deno.land/x/postgresjs@v3.4.4/polyfills.js:138:16)
    at error (https://deno.land/x/postgresjs@v3.4.4/polyfills.js:131:5)
    at eventLoopTick (ext:core/01_core.js:168:7)

[alexgleason]

So I edited the code manually and logged it. err is:

InvalidData: invalid peer certificate: UnknownIssuer
    at async TlsConn.read (ext:deno_net/01_net.js:150:15)
    at async success (https://raw.githubusercontent.com/alexgleason/postgres.js/51ac395234089d5b7806c471830a1a472958fb06/deno/polyfills.js:97:56) {
  name: "InvalidData"
}

And the value of query is... true ???

EDIT: It's because query is being set to true here:

postgres/src/connection.js

Line 112 in f58cd4f

initial = query || true

[marbemac]

Running into this as well

[patrickReiis]

I'm also having this problem

[dhardtke]

We are also seeing this error in our projects:

TypeError: Cannot read properties of undefined (reading 'replace')
    at queryError (/usr/src/app/node_modules/postgres/cjs/src/connection.js:389:48)
    at errored (/usr/src/app/node_modules/postgres/cjs/src/connection.js:383:14)
    at TLSSocket.data (/usr/src/app/node_modules/postgres/cjs/src/connection.js:318:9)
    at TLSSocket.emit (node:events:519:28)
    at addChunk (node:internal/streams/readable:559:12)
    at readableAddChunkPushByteMode (node:internal/streams/readable:510:3)
    at Readable.push (node:internal/streams/readable:390:5)
    at TLSWrap.onStreamRead (node:internal/stream_base_commons:191:23)

I am not convinced it is an issue with the initial query since /usr/src/app/node_modules/postgres/cjs/src/connection.js:383:14 points to

postgres/src/connection.js

Line 383 in a2c7de1

query && queryError(query, err)

It is very problematic that we don't see the actual error. It does not happen on startup but when our NodeJS backend is under high load.

[arenddeboer]

I'm getting this error with target_session_attrs: 'primary' where the replica is down and the replica as first host in a multi host connection string. Will try to dig deeper.

[alexgleason]

If you are using Deno you can work around this by switching to my fork with:

import postgres from "https://gitlab.com/soapbox-pub/postgres.js/-/raw/e79d7d2039446fbf7a37d4eca0d17e94a94b8b53/deno/mod.js";

I've been using this fork in a high workload application for 3 or 4 months now.

It's just the patch applied from #944

If you're on Node, perhaps edit the file in node_modules manually to apply the change from #944. That will at least give you the correct error message, which you may only need temporarily to solve the issue.

[arenddeboer]

Thanks @alexgleason
I'm on node.js and the timeout seems to trigger connectTimedOut() which calls errored() which calls queryError().
As this only happens during startup and the reconnecting mechanism to other hosts in the host string works at runtime, I will resort to sorting the host string with the primary first before initializing.