diff options
author | Jenkins <jenkins@review.openstack.org> | 2016-09-14 20:45:43 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2016-09-14 20:45:43 +0000 |
commit | cacb3a3a1501a930a05b8494d6d1215fe96cf7af (patch) | |
tree | a7676ec77635b1bff0e7e2858dd6e048d3ee66d2 /swiftclient/shell.py | |
parent | 5eb0f3debc320340965460c04a1d6ca4f9c423a3 (diff) | |
parent | 4c955751d340a8f71a2eebdb3c58d90b36874a66 (diff) | |
download | python-swiftclient-cacb3a3a1501a930a05b8494d6d1215fe96cf7af.tar.gz |
Merge "Make tempurl command check for valid object path"
Diffstat (limited to 'swiftclient/shell.py')
-rwxr-xr-x | swiftclient/shell.py | 14 |
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: |