From cc1c79b27d01d5bc9d93b62d1afc368386f56319 Mon Sep 17 00:00:00 2001 From: Cheick Keita Date: Fri, 29 Jan 2021 12:32:17 -0800 Subject: [PATCH] Fix bug when checking if the default cli application exists (#488) --- src/deployment/deploy.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/deployment/deploy.py b/src/deployment/deploy.py index db00564a7..8c36bf8cd 100644 --- a/src/deployment/deploy.py +++ b/src/deployment/deploy.py @@ -328,11 +328,11 @@ class Client: (password_id, password) = self.create_password(app.object_id) - cli_app = client.applications.list(filter="appId eq '%s'" % ONEFUZZ_CLI_APP) + cli_app = list( + client.applications.list(filter="appId eq '%s'" % ONEFUZZ_CLI_APP) + ) - onefuzz_cli_app_uuid = uuid.UUID(ONEFUZZ_CLI_APP) - - if not cli_app: + if len(cli_app) == 0: logger.info( "Could not find the default CLI application under the current " "subscription, creating a new one" @@ -346,7 +346,7 @@ class Client: } else: - authorize_application(onefuzz_cli_app_uuid, app.app_id) + authorize_application(uuid.UUID(ONEFUZZ_CLI_APP), app.app_id) self.results["client_id"] = app.app_id self.results["client_secret"] = password