From d7060590062541479001bce48a39d0f770d479da Mon Sep 17 00:00:00 2001 From: Andrew Klychkov Date: Sat, 14 Dec 2019 07:16:27 +0300 Subject: [PATCH] Backport of 65787 - postgresql_sequence: use query parameters with cursor object --- ...-postgresql_sequence_use_query_params_with_cursor.yml | 2 ++ .../modules/database/postgresql/postgresql_sequence.py | 9 +++++---- 2 files changed, 7 insertions(+), 4 deletions(-) create mode 100644 changelogs/fragments/65787-postgresql_sequence_use_query_params_with_cursor.yml diff --git a/changelogs/fragments/65787-postgresql_sequence_use_query_params_with_cursor.yml b/changelogs/fragments/65787-postgresql_sequence_use_query_params_with_cursor.yml new file mode 100644 index 00000000000..27c2bc17a36 --- /dev/null +++ b/changelogs/fragments/65787-postgresql_sequence_use_query_params_with_cursor.yml @@ -0,0 +1,2 @@ +bugfixes: +- postgresql_sequence - use query parameters with cursor object (https://github.com/ansible/ansible/pull/65787). diff --git a/lib/ansible/modules/database/postgresql/postgresql_sequence.py b/lib/ansible/modules/database/postgresql/postgresql_sequence.py index 095bd1f4c6a..1ca0c0c1a8f 100644 --- a/lib/ansible/modules/database/postgresql/postgresql_sequence.py +++ b/lib/ansible/modules/database/postgresql/postgresql_sequence.py @@ -369,11 +369,12 @@ class Sequence(object): "LEFT JOIN pg_namespace n ON n.oid = c.relnamespace " "WHERE NOT pg_is_other_temp_schema(n.oid) " "AND c.relkind = 'S'::\"char\" " - "AND sequence_name = '%s' " - "AND sequence_schema = '%s'" % (self.name, - self.schema)) + "AND sequence_name = %(name)s " + "AND sequence_schema = %(schema)s") - res = exec_sql(self, query, add_to_executed=False) + res = exec_sql(self, query, + query_params={'name': self.name, 'schema': self.schema}, + add_to_executed=False) if not res: self.exists = False