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>
Reviewed-on: http://review.gluster.org/6664
Tested-by: Luis Pabon <lpabon@redhat.com>

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'):