summaryrefslogtreecommitdiff
path: root/swiftclient/shell.py
diff options
context:
space:
mode:
authorJenkins <jenkins@review.openstack.org>2016-09-14 20:45:43 +0000
committerGerrit Code Review <review@openstack.org>2016-09-14 20:45:43 +0000
commitcacb3a3a1501a930a05b8494d6d1215fe96cf7af (patch)
treea7676ec77635b1bff0e7e2858dd6e048d3ee66d2 /swiftclient/shell.py
parent5eb0f3debc320340965460c04a1d6ca4f9c423a3 (diff)
parent4c955751d340a8f71a2eebdb3c58d90b36874a66 (diff)
downloadpython-swiftclient-cacb3a3a1501a930a05b8494d6d1215fe96cf7af.tar.gz
Merge "Make tempurl command check for valid object path"
Diffstat (limited to 'swiftclient/shell.py')
-rwxr-xr-xswiftclient/shell.py14
1 files changed, 7 insertions, 7 deletions
diff --git a/swiftclient/shell.py b/swiftclient/shell.py
index dc42e52..9386795 100755
--- a/swiftclient/shell.py
+++ b/swiftclient/shell.py
@@ -1240,17 +1240,17 @@ def st_tempurl(parser, args, thread_manager):
parsed = urlparse(path)
- try:
- seconds = int(seconds)
- except ValueError:
- thread_manager.error('Seconds must be an integer')
- return
if method.upper() not in ['GET', 'PUT', 'HEAD', 'POST', 'DELETE']:
thread_manager.print_msg('WARNING: Non default HTTP method %s for '
'tempurl specified, possibly an error' %
method.upper())
- path = generate_temp_url(parsed.path, seconds, key, method,
- absolute=options['absolute_expiry'])
+ try:
+ path = generate_temp_url(parsed.path, seconds, key, method,
+ absolute=options['absolute_expiry'])
+ except ValueError as err:
+ thread_manager.error(err)
+ return
+
if parsed.scheme and parsed.netloc:
url = "%s://%s%s" % (parsed.scheme, parsed.netloc, path)
else: