Skip to content

Commit d8c0f3b

Browse files
author
Jussi Kukkonen
committed
Metadata API: Be more careful with container args
If argument is an empty container, we want to use the given empty container. Only create a new container if argument is None. Signed-off-by: Jussi Kukkonen <jkukkonen@vmware.com>
1 parent 4392574 commit d8c0f3b

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

tuf/api/metadata.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,7 @@ def __init__(
124124
unrecognized_fields: Optional[Mapping[str, Any]] = None,
125125
):
126126
self.signed: T = signed
127-
self.signatures = signatures or {}
127+
self.signatures = signatures if signatures is not None else {}
128128
self.unrecognized_fields: Mapping[str, Any] = unrecognized_fields or {}
129129

130130
def __eq__(self, other: Any) -> bool:
@@ -854,7 +854,7 @@ def __init__(
854854
):
855855
super().__init__(version, spec_version, expires, unrecognized_fields)
856856
self.consistent_snapshot = consistent_snapshot
857-
self.keys = keys or {}
857+
self.keys = keys if keys is not None else {}
858858

859859
if roles is None:
860860
roles = {r: Role([], 1) for r in TOP_LEVEL_ROLE_NAMES}
@@ -1676,7 +1676,7 @@ def __init__(
16761676
unrecognized_fields: Optional[Mapping[str, Any]] = None,
16771677
) -> None:
16781678
super().__init__(version, spec_version, expires, unrecognized_fields)
1679-
self.targets = targets or {}
1679+
self.targets = targets if targets is not None else {}
16801680
self.delegations = delegations
16811681

16821682
def __eq__(self, other: Any) -> bool:

0 commit comments

Comments
 (0)