Merge pull request #690 from thekad/feature/clearer_secret_manager_message

Clear up error responses from gcp_secret_manager lookup
This commit is contained in:
Chris Hawk 2025-06-20 13:17:42 -07:00 committed by GitHub
commit 4eec57c4e1
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -204,8 +204,23 @@ class LookupModule(LookupBase):
)
response = auth.get(url)
self._display.vvv(msg=f"List Version Response: {response.status_code} for {response.request.url}: {response.json()}")
if response.status_code != 200:
self.raise_error(module, f"unable to list versions of secret {response.status_code}")
if response.status_code >= 500: # generic server error
self.raise_error(
module,
f"server error encountered while looking for secret '{module.params['name']}', code: {response.status_code}"
)
elif response.status_code >= 400: # generic client request error
self.raise_error(
module,
f"client error encountered while looking for secret '{module.params['name']}', code: {response.status_code}"
)
elif response.status_code >= 300: # all other possible errors
self.raise_error(
module,
f"unable to list versions for secret '{module.params['name']}', code: {response.status_code}"
)
else:
pass
version_list = response.json()
if "versions" in version_list:
versions_numbers = []