[Tile-serving] [openstreetmap/osm2pgsql] osm2pgsql-replication will now use postgresql's application_name (PR #1980)

[Paul Norman]

@pnorman requested changes on this pull request.



> @@ -80,10 +80,10 @@ def connect(args):
     """
     # If dbname looks like a conninfo string use it as such
     if args.database and any(part in args.database for part in ['=', '://']):
-        return psycopg.connect(args.database)
+        return psycopg.connect(args.database, application_name="osm2pgsql-replication")

I'd rather set `fallback_application_name` so that a user can potentially override with `PGAPPNAME` if they're running multiple DBs

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/osm2pgsql/pull/1980#pullrequestreview-1495647114
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/osm2pgsql/pull/1980/review/1495647114 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/tile-serving/attachments/20230623/bcbf1edb/attachment.htm>