summaryrefslogtreecommitdiffstats
path: root/gluster/swift/common/middleware/gswauth/bin/gswauth-delete-user
diff options
context:
space:
mode:
authorThiago da Silva <thiago@redhat.com>2013-12-20 13:23:35 -0500
committerLuis Pabon <lpabon@redhat.com>2014-01-07 19:32:17 -0800
commitac55cfb30b8600a105575783f26c1fe2048407af (patch)
tree3a82199ea09577d768733ff7dba7a16956bfaf34 /gluster/swift/common/middleware/gswauth/bin/gswauth-delete-user
parent94a3f539e75b069bb1f9df6f850adfe16d76b572 (diff)
fixed gswauth cmd tools to require admin key argument
All tools were changed to display the usage help text in case the admin key is not provided as an argument Bug 1261677: https://bugs.launchpad.net/gluster-swift/+bug/1261677 Change-Id: I37fa8518be970364b81955644931f6e416bcdc42 Signed-off-by: Thiago da Silva <thiago@redhat.com> Reviewed-on: http://review.gluster.org/6553 Reviewed-by: Prashanth Pai <ppai@redhat.com> Tested-by: Prashanth Pai <ppai@redhat.com> Reviewed-by: pushpesh sharma <psharma@redhat.com> Tested-by: pushpesh sharma <psharma@redhat.com> Reviewed-by: Luis Pabon <lpabon@redhat.com>
Diffstat (limited to 'gluster/swift/common/middleware/gswauth/bin/gswauth-delete-user')
-rwxr-xr-xgluster/swift/common/middleware/gswauth/bin/gswauth-delete-user8
1 files changed, 6 insertions, 2 deletions
diff --git a/gluster/swift/common/middleware/gswauth/bin/gswauth-delete-user b/gluster/swift/common/middleware/gswauth/bin/gswauth-delete-user
index 2b4f4fe..f778603 100755
--- a/gluster/swift/common/middleware/gswauth/bin/gswauth-delete-user
+++ b/gluster/swift/common/middleware/gswauth/bin/gswauth-delete-user
@@ -30,16 +30,20 @@ if __name__ == '__main__':
default='http://127.0.0.1:8080/auth/', help='The URL to the auth '
'subsystem (default: http://127.0.0.1:8080/auth/')
parser.add_option('-U', '--admin-user', dest='admin_user',
- default='.super_admin', help='The user with admin rights to add users '
+ default='.super_admin',
+ help='The user with admin rights to delete users '
'(default: .super_admin).')
parser.add_option('-K', '--admin-key', dest='admin_key',
- help='The key for the user with admin rights to add users.')
+ help='The key for the user with admin rights to delete '
+ 'users is required.')
args = argv[1:]
if not args:
args.append('-h')
(options, args) = parser.parse_args(args)
if len(args) != 2:
parser.parse_args(['-h'])
+ if options.admin_key is None:
+ parser.parse_args(['-h'])
account, user = args
parsed = urlparse(options.admin_url)
if parsed.scheme not in ('http', 'https'):