Check for auth URL before password (bug 1076235)
Change-Id: I9cebbf199e8cf3d9dd7de532b30da9f732a6dab1
This commit is contained in:
		| @@ -300,6 +300,11 @@ class OpenStackIdentityShell(object): | |||||||
|                         'Expecting a username provided via either ' |                         'Expecting a username provided via either ' | ||||||
|                         '--os-username or env[OS_USERNAME]') |                         '--os-username or env[OS_USERNAME]') | ||||||
|  |  | ||||||
|  |                 if not args.os_auth_url: | ||||||
|  |                     raise exc.CommandError( | ||||||
|  |                         'Expecting an auth URL via either --os-auth-url or ' | ||||||
|  |                         'env[OS_AUTH_URL]') | ||||||
|  |  | ||||||
|                 if not args.os_password: |                 if not args.os_password: | ||||||
|                     # No password, If we've got a tty, try prompting for it |                     # No password, If we've got a tty, try prompting for it | ||||||
|                     if hasattr(sys.stdin, 'isatty') and sys.stdin.isatty(): |                     if hasattr(sys.stdin, 'isatty') and sys.stdin.isatty(): | ||||||
| @@ -316,11 +321,6 @@ class OpenStackIdentityShell(object): | |||||||
|                             '--os-password, env[OS_PASSWORD], or ' |                             '--os-password, env[OS_PASSWORD], or ' | ||||||
|                             'prompted response') |                             'prompted response') | ||||||
|  |  | ||||||
|                 if not args.os_auth_url: |  | ||||||
|                     raise exc.CommandError( |  | ||||||
|                         'Expecting an auth URL via either --os-auth-url or ' |  | ||||||
|                         'env[OS_AUTH_URL]') |  | ||||||
|  |  | ||||||
|             # if it looks like the user wants to provide a service token |             # if it looks like the user wants to provide a service token | ||||||
|             # but is missing something |             # but is missing something | ||||||
|             if args.os_token or args.os_endpoint and not ( |             if args.os_token or args.os_endpoint and not ( | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user
	 Dolph Mathews
					Dolph Mathews