Skip to content

Commit 045606e

Browse files
authored
Merge pull request #153 from CHIMEFRB/fix-invalid-cert-handling-3626286325252793721
🧹 Refactor invalid certificate handling in cadcclient
2 parents 0c83dd1 + 4468772 commit 045606e

1 file changed

Lines changed: 6 additions & 3 deletions

File tree

dtcli/utilities/cadcclient.py

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -70,10 +70,13 @@ def _connect(
7070
storage = StorageInventoryClient(cert, resource_id=storage_resource_id)
7171
query = CadcTapClient(cert, resource_id=query_resource_id)
7272
return cert, storage, query
73-
# TODO: Handle invalid cert
7473
except ValueError as error:
75-
logger.error(error)
76-
raise error
74+
logger.error(
75+
"Authorization failed: The provided CANFAR certificate is "
76+
"invalid or expired. Please ensure you have a valid "
77+
"certificate and try again."
78+
)
79+
raise ValueError("Invalid or expired CANFAR certificate.") from error
7780

7881

7982
def get(

0 commit comments

Comments
 (0)