Skip to content
Snippets Groups Projects
Commit 42ed3e12 authored by vlorentz's avatar vlorentz
Browse files

Fix log level + status code of the client CLI in case of error.

'return' statements were ignored, and log level was too low for an error.
parent eb2631a7
No related branches found
No related tags found
No related merge requests found
......@@ -5,6 +5,7 @@
import os
import logging
import sys
import tempfile
import uuid
......@@ -335,12 +336,8 @@ https://docs.softwareheritage.org/devel/swh-deposit/getting-started.html.
metadata_deposit, collection, slug, partial, deposit_id,
replace, url, name, author)
except InputError as e:
msg = 'Problem during parsing options: %s' % e
r = {
'error': msg,
}
logger.info(r)
return 1
logger.error('Problem during parsing options: %s', e)
sys.exit(1)
try:
if verbose:
......@@ -382,12 +379,8 @@ def status(ctx, url, username, password, deposit_id):
client = _client(url, username, password)
collection = _collection(client)
except InputError as e:
msg = 'Problem during parsing options: %s' % e
r = {
'error': msg,
}
logger.info(r)
return 1
logger.error('Problem during parsing options: %s', e)
sys.exit(1)
r = client.deposit_status(
collection=collection, deposit_id=deposit_id)
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment