Make GetRepo, handle missing repo options, with out breaking

manatools · Oct 1, 2021 · a8f3591 · a8f3591
1 parent 5bc8e23
commit a8f3591
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/python/dnfdaemon/server/__init__.py b/python/dnfdaemon/server/__init__.py
@@ -346,8 +346,14 @@ def get_repo(self, repo_id):
         value = json.dumps(None)
         repo = self.base.repos.get(repo_id, None)  # get the repo object
         if repo:
-            # VectorString is not JSON serializable let's convert to list
-            repo_conf = dict([(c, getattr(repo, c) if 'VectorString' not in str(type(getattr(repo, c))) else list(getattr(repo, c))) for c in optionKeys])
+            repo_conf = {}
+            for c in optionKeys:
+                if hasattr(repo,c):
+                    value = getattr(repo, c)
+                    # VectorString is not JSON serializable let's convert to list
+                    if 'VectorString' in str(type(value)):
+                        value = list(value)
+                    repo_conf[c] = value
 
             enab = repo.enabled
             if not enab: