diff options
| author | Jonathan de Jong <jonathan@automatia.nl> | 2021-04-08 23:38:54 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-04-08 22:38:54 +0100 |
| commit | 2ca4e349e9d0c606d802ae15c06089080fa4f27e (patch) | |
| tree | 0b3e8448124e0c0cbe128b27ab1641be02697536 /synapse/federation | |
| parent | Merge pull request #9766 from matrix-org/rav/drop_py35 (diff) | |
| download | synapse-2ca4e349e9d0c606d802ae15c06089080fa4f27e.tar.xz | |
Bugbear: Add Mutable Parameter fixes (#9682)
Part of #9366 Adds in fixes for B006 and B008, both relating to mutable parameter lint errors. Signed-off-by: Jonathan de Jong <jonathan@automatia.nl>
Diffstat (limited to 'synapse/federation')
| -rw-r--r-- | synapse/federation/units.py | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/synapse/federation/units.py b/synapse/federation/units.py index b662c42621..0f8bf000ac 100644 --- a/synapse/federation/units.py +++ b/synapse/federation/units.py @@ -18,6 +18,7 @@ server protocol. """ import logging +from typing import Optional import attr @@ -98,7 +99,7 @@ class Transaction(JsonEncodedObject): "pdus", ] - def __init__(self, transaction_id=None, pdus=[], **kwargs): + def __init__(self, transaction_id=None, pdus: Optional[list] = None, **kwargs): """If we include a list of pdus then we decode then as PDU's automatically. """ @@ -107,7 +108,7 @@ class Transaction(JsonEncodedObject): if "edus" in kwargs and not kwargs["edus"]: del kwargs["edus"] - super().__init__(transaction_id=transaction_id, pdus=pdus, **kwargs) + super().__init__(transaction_id=transaction_id, pdus=pdus or [], **kwargs) @staticmethod def create_new(pdus, **kwargs): |