Fix bug in basic authentication

Change-Id: If3f8231b5a3330009379d6d20d47f0be895ecdc8
Issue-ID: CCSDK-425
Signed-off-by: jh245g <jh245g@att.com>

diff --git a/helm/plugin/tasks.py b/helm/plugin/tasks.py
index 5ff7df8..4ffcd81 100644 (file)
--- a/helm/plugin/tasks.py
+++ b/helm/plugin/tasks.py
@@ -220,7 +220,7 @@ def config(**kwargs):
     if configJson == '' and configUrl == '':
         ctx.logger.debug("Will use default HELM value")
     elif configJson == '' and configUrl != '':
-        if configUrl.find("@"):
+        if configUrl.find("@") != -1:
             head, end = configUrl.rsplit('@', 1)
             head, auth = head.rsplit('//', 1)
             configUrl = head + '//' + end

diff --git a/helm/plugin/workflows.py b/helm/plugin/workflows.py
index 9870bdf..2999b73 100644 (file)
--- a/helm/plugin/workflows.py
+++ b/helm/plugin/workflows.py
@@ -38,7 +38,7 @@ def upgrade(node_instance_id, config_json, config_url, config_format,
     if config_json == '' and config_url == '':
         kwargs['config'] = config_json
     elif config_json == '' and config_url != '':
-        if config_url.find("@"):
+        if config_url.find("@") != -1:
             head, end = config_url.rsplit('@', 1)
             head, auth = head.rsplit('//', 1)
             config_url = head + '//' + end