Elvis Pranskevichus
Elvis Pranskevichus
Oh, right, looks like plain `quote()` isn't quoting enough, use `urllib.parse.quote_plus()`. I'll amend the docs.
Oh, it seems like Postgres has crashed in this run. Does this happen consistently?
> I've created minimal reproducible example [here](https://github.com/Pentusha/edgedb_setup_crush_demo/actions/runs/2392403326). Thanks for repro!
`SHOULD_RETRY` should be inherited, and it's set on the parent `TransactionConflictError`. The bindings might not be reflecting that faithfully, though.
What happens is that `anyio` cancels `await conn.close()` before it has a chance to send the termination message to Postgres. Connection does get closed from the client side, but Postgres...
Good point! We need to allow specifying the `setup` counterpart in the pool configuration.
As a note, it is often better to allocate a dedicated persistent connection for the purpose of listening to notifications so that you don't suffer the overhead of `LISTEN`/`UNLISTEN` on...
`RAISE NOTICE` is different from `NOTIFY`. For the former you need `add_log_listener()`, and for the latter its `add_listener()`.
It's possible in principle ([gel](https://docs.geldata.com/) supports this) but would require a non-trivial amount of work.
The `insecure` setting should NOT actually drop HTTPS altogether. Instead, it should be ignoring TLS errors. The only place where not doing HTTPS makes sense is in frontend code where...