From 815bc8d1bf8f0811d7583ffeebfad463cb481354 Mon Sep 17 00:00:00 2001 From: Sanne Raymaekers Date: Fri, 3 Sep 2021 19:27:10 +0200 Subject: [PATCH] util/rhsm: Check if repositories is None before iterating When `get_fallback_rhsm_secrets` was used, `Subscriptions.repositories` was None, and `get_secrets` never returned the fallback secrets. So check if `repositories` is None before iterating over it, otherwise return the fallback secrets. --- osbuild/util/rhsm.py | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/osbuild/util/rhsm.py b/osbuild/util/rhsm.py index 21a2d50..3ab1729 100644 --- a/osbuild/util/rhsm.py +++ b/osbuild/util/rhsm.py @@ -93,13 +93,14 @@ class Subscriptions: def get_secrets(self, url): # Try to find a matching URL from redhat.repo file first - for parameters in self.repositories.values(): - if parameters["matchurl"].match(url) is not None: - return { - "ssl_ca_cert": parameters["sslcacert"], - "ssl_client_key": parameters["sslclientkey"], - "ssl_client_cert": parameters["sslclientcert"] - } + if self.repositories is not None: + for parameters in self.repositories.values(): + if parameters["matchurl"].match(url) is not None: + return { + "ssl_ca_cert": parameters["sslcacert"], + "ssl_client_key": parameters["sslclientkey"], + "ssl_client_cert": parameters["sslclientcert"] + } # In case there is no matching URL, try the fallback if self.secrets: -- 2.21.1 (Apple Git-122.3)