From 9616ae8068814cd07e5e935827e84930921be118 Mon Sep 17 00:00:00 2001 From: Jon Turney Date: Thu, 6 Apr 2023 16:58:43 +0100 Subject: Improve error handling for untest|vault with invalid package name Produce an error message for untest|vault with invalid package name, rather than a python backtrace. --- calm/tool_util.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/calm/tool_util.py b/calm/tool_util.py index f677f86..dc4d882 100644 --- a/calm/tool_util.py +++ b/calm/tool_util.py @@ -48,8 +48,12 @@ def permitted(p): logging.error("who are you?") return False - # CYGNAME is a maintainer for package pkg_list = maintainers.pkg_list(common_constants.PKGMAINT) + if p not in pkg_list: + logging.error("%s is not a package name" % p) + return False + + # CYGNAME is a maintainer for package if cygname in pkg_list[p].maintainers(): return True -- cgit v1.2.3