Don't call 'encode_parameter' no-op

synapse/storage/_base.py

@@ -167,10 +167,6 @@ class LoggingTransaction(object):
         sql = self.database_engine.convert_param_style(sql)
 
         if args and args[0]:
-            args = list(args)
-            args[0] = [
-                self.database_engine.encode_parameter(a) for a in args[0]
-            ]
             try:
                 sql_logger.debug(
                     "[SQL values] {%s} " + ", ".join(("<%r>",) * len(args[0])),

synapse/storage/engines/postgres.py

@@ -36,9 +36,6 @@ class PostgresEngine(object):
     def convert_param_style(self, sql):
         return sql.replace("?", "%s")
 
-    def encode_parameter(self, param):
-        return param
-
     def on_new_connection(self, db_conn):
         db_conn.set_isolation_level(
             self.module.extensions.ISOLATION_LEVEL_REPEATABLE_READ

synapse/storage/engines/sqlite3.py

@@ -26,9 +26,6 @@ class Sqlite3Engine(object):
     def convert_param_style(self, sql):
         return sql
 
-    def encode_parameter(self, param):
-        return param
-
     def on_new_connection(self, db_conn):
         self.prepare_database(db_conn)