From 468a380ea44d6277bfa90190f9db9580992d9dc8 Mon Sep 17 00:00:00 2001 From: Joshua Lock Date: Wed, 25 Mar 2020 11:25:20 +0000 Subject: [PATCH] Fix replacing targets in add_targets() Match the pattern in add_target() where if the filepath already exists in roleinfo['paths'] it is updated to replace the existing entry with the new fileinfo. Signed-off-by: Joshua Lock --- tuf/repository_tool.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tuf/repository_tool.py b/tuf/repository_tool.py index 49e9d45b..dd88159d 100755 --- a/tuf/repository_tool.py +++ b/tuf/repository_tool.py @@ -2027,10 +2027,10 @@ def add_targets(self, list_of_targets): for relative_target in relative_list_of_targets: if relative_target not in roleinfo['paths']: logger.debug('Adding new target: ' + repr(relative_target)) - roleinfo['paths'].update({relative_target: {}}) else: logger.debug('Replacing target: ' + repr(relative_target)) + roleinfo['paths'].update({relative_target: {}}) tuf.roledb.update_roleinfo(self.rolename, roleinfo, repository_name=self._repository_name)