diff options
author | Rob Austein <sra@hactrn.net> | 2016-06-09 00:28:58 +0000 |
---|---|---|
committer | Rob Austein <sra@hactrn.net> | 2016-06-09 00:28:58 +0000 |
commit | a656fbaeaff90a2116c86caa570b705a6078312b (patch) | |
tree | 0758a71d0777946e348fc4cb6ada3db3b5b318ed /rp/config | |
parent | 773e2d868de090f1e98de89f4afd60942084a755 (diff) |
Fix badly written XXX_Driver.execute() methods: calling sequence is
specified by DB API, just use it, rather than getting clever with
variable-length argument sequences.
svn path=/branches/tk705/; revision=6442
Diffstat (limited to 'rp/config')
-rwxr-xr-x | rp/config/rpki-sql-setup | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/rp/config/rpki-sql-setup b/rp/config/rpki-sql-setup index 9c88449f..eb9bb153 100755 --- a/rp/config/rpki-sql-setup +++ b/rp/config/rpki-sql-setup @@ -89,7 +89,7 @@ class MySQL_Driver(Abstract_Driver): self.execute("SELECT COUNT(*) FROM information_schema.tables WHERE table_schema = '{0.database}'".format(udb)) return bool(self.fetchone()[0]) - def execute(self, *args): + def execute(self, query, args = None): try: self._cur except AttributeError: @@ -105,8 +105,8 @@ class MySQL_Driver(Abstract_Driver): passwd = getpass.getpass("Please enter your MySQL root password: ")) self._db.autocommit(True) self._cur = self._db.cursor() - self.log("MySQL driver executing {}".format(", ".join(args))) - return self._cur.execute(*args) + self.log("MySQL driver executing {!r} {!r}".format(query, args)) + return self._cur.execute(query, args) def create(self, udb): self.execute("CREATE DATABASE IF NOT EXISTS {0.database}".format(udb)) @@ -179,7 +179,7 @@ class PostgreSQL_Driver(Abstract_Driver): os.seteuid(new_uid) return old_uid - def execute(self, *args): + def execute(self, query, args = None): try: self._cur except AttributeError: @@ -193,8 +193,8 @@ class PostgreSQL_Driver(Abstract_Driver): finally: if self._pw is not None: self._seteuid(euid) - self.log("PostgreSQL driver executing {}".format(", ".join(args))) - return self._cur.execute(*args) + self.log("PostgreSQL driver executing {!r} {!r}".format(query, args)) + return self._cur.execute(query, args) def _db_accessible_test(self, udb): pw = pwd.getpwnam(udb.username) |