From b85198f0e4db86b2bad744de372887aec6640fe5 Mon Sep 17 00:00:00 2001 From: Stu Tomlinson Date: Thu, 10 Nov 2022 12:25:07 +0000 Subject: [PATCH] pgsql: do not set standard_conforming_strings to 'off' Since f9120f75b0 string literals use escape string syntax (E'...') on pgsql, so it is no longer necessary to set standard_conforming_strings to 'off' Fixes #3932 --- src/ejabberd_sql.erl | 24 ++++++++---------------- 1 file changed, 8 insertions(+), 16 deletions(-) diff --git a/src/ejabberd_sql.erl b/src/ejabberd_sql.erl index 39c2eab7d..446f4951f 100644 --- a/src/ejabberd_sql.erl +++ b/src/ejabberd_sql.erl @@ -1005,22 +1005,14 @@ sqlite_to_odbc(_Host, _) -> %% Open a database connection to PostgreSQL pgsql_connect(Server, Port, DB, Username, Password, ConnectTimeout, Transport, SSLOpts) -> - case pgsql:connect([{host, Server}, - {database, DB}, - {user, Username}, - {password, Password}, - {port, Port}, - {transport, Transport}, - {connect_timeout, ConnectTimeout}, - {as_binary, true}|SSLOpts]) of - {ok, Ref} -> - pgsql:squery(Ref, [<<"alter database \"">>, DB, <<"\" set ">>, - <<"standard_conforming_strings='off';">>]), - pgsql:squery(Ref, [<<"set standard_conforming_strings to 'off';">>]), - {ok, Ref}; - Err -> - Err - end. + pgsql:connect([{host, Server}, + {database, DB}, + {user, Username}, + {password, Password}, + {port, Port}, + {transport, Transport}, + {connect_timeout, ConnectTimeout}, + {as_binary, true}|SSLOpts]). %% Convert PostgreSQL query result to Erlang ODBC result formalism pgsql_to_odbc({ok, PGSQLResult}) ->