Apply suggestions from code review

Co-authored-by: Joshua Lock <jlock@vmware.com>
theupdateframework · Jul 23, 2020 · 8fe9ef7 · 8fe9ef7
1 parent 494d570
commit 8fe9ef7
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/tuf/client/updater.py b/tuf/client/updater.py
@@ -954,7 +954,7 @@ def _import_delegations(self, parent_role):
           key, _ = securesystemslib.keys.format_metadata_to_key(keyinfo, keyid)
 
           key['keyid'] = keyid
-          tuf.keydb.add_key(key, keyid=None, repository_name=self.repository_name)
+          tuf.keydb.add_key(key, repository_name=self.repository_name)
 
         except tuf.exceptions.KeyAlreadyExistsError:
           pass

diff --git a/tuf/keydb.py b/tuf/keydb.py
@@ -128,7 +128,7 @@ def create_keydb_from_root_metadata(root_metadata, repository_name='default'):
       # keyids, otherwise add_key() will have no reference to it.
       try:
         key_dict['keyid'] = keyid
-        add_key(key_dict, keyid=None, repository_name=repository_name)
+        add_key(key_dict, repository_name=repository_name)
 
       # Although keyid duplicates should *not* occur (unique dict keys), log a
       # warning and continue.  However, 'key_dict' may have already been