Improved error message for ACS

Improved error message for Alternate Content Source refresh when it has insufficient permissions. closes: pulp#2667 pulp#2667
pavelpicka · Jul 28, 2022 · d905545 · d905545
1 parent a62aaf1
commit d905545
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/CHANGES/2667.bugfix b/CHANGES/2667.bugfix
@@ -0,0 +1 @@
+Improved error message for Alternate Content Source refresh when it has insufficient permissions.
diff --git a/pulp_rpm/app/tasks/synchronizing.py b/pulp_rpm/app/tasks/synchronizing.py
@@ -293,6 +293,13 @@ def normalize_url(url_to_normalize):
         # If 'custom_url' is passed it is a call from ACS refresh
         # which doesn't support mirror lists.
         if custom_url:
+            if exc.status >= 400:
+                raise ValueError(
+                    _(
+                        "ACS remote for url '{}' raised an error '{} {}'. "
+                        "Please check your ACS remote configuration."
+                    ).format(custom_url, exc.status, exc.message)
+                )
             raise ValueError(
                 _("Remote URL {} for Alternate Content Source is invalid").format(custom_url)
             )