From 31116ed030d0c413b086b77bf064a370db862057 Mon Sep 17 00:00:00 2001 From: Ian Wienand Date: Thu, 2 Aug 2018 19:17:31 +1000 Subject: [PATCH] Revert "Debugging for readthedoc web ping" This reverts commit 5a38208e3b3899231a3a813639498843676bdade. Turns out it was a badly encoded password; see https://review.openstack.org/588182 Change-Id: Ib6c1d3c13340c1cacbc5c6b7e0a77841c0d78afb --- roles/trigger-readthedocs/tasks/main.yaml | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/roles/trigger-readthedocs/tasks/main.yaml b/roles/trigger-readthedocs/tasks/main.yaml index 108d67f44..59e11562d 100644 --- a/roles/trigger-readthedocs/tasks/main.yaml +++ b/roles/trigger-readthedocs/tasks/main.yaml @@ -15,12 +15,6 @@ msg: password is required when using rtd_credentials.username when: rtd_credentials.password is not defined - - debug: - msg: 'https://readthedocs.org/api/v2/webhook/{{ rtd_project_name}}/{{ rtd_webhook_id}}/' - - - debug: - msg: 'username <{{ rtd_credentials.username }}>' - - name: Trigger readthedocs build webhook via authentication uri: method: POST @@ -30,6 +24,8 @@ # NOTE(ianw): testing it seems the API doesn't respond with # 401 so this is required force_basic_auth: yes + # avoid logging any credentials + no_log: true - when: rtd_credentials.integration_token is defined and rtd_credentials.username is not defined