From patchwork Thu Aug 30 21:06:12 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Michael-Doyle Hudson X-Patchwork-Id: 11070 Return-Path: X-Original-To: patchwork@peony.canonical.com Delivered-To: patchwork@peony.canonical.com Received: from fiordland.canonical.com (fiordland.canonical.com [91.189.94.145]) by peony.canonical.com (Postfix) with ESMTP id 8D6A123E52 for ; Thu, 30 Aug 2012 21:06:15 +0000 (UTC) Received: from mail-ie0-f180.google.com (mail-ie0-f180.google.com [209.85.223.180]) by fiordland.canonical.com (Postfix) with ESMTP id E129AA192C9 for ; Thu, 30 Aug 2012 21:05:38 +0000 (UTC) Received: by ieak11 with SMTP id k11so1127166iea.11 for ; Thu, 30 Aug 2012 14:06:14 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=x-forwarded-to:x-forwarded-for:delivered-to:received-spf :content-type:mime-version:x-launchpad-project:x-launchpad-branch :x-launchpad-message-rationale:x-launchpad-branch-revision-number :x-launchpad-notification-type:to:from:subject:message-id:date :reply-to:sender:errors-to:precedence:x-generated-by :x-launchpad-hash:x-gm-message-state; bh=/37odkjj28tg4DufmdpaAczT5xgYLhs1+J1bV+0WjHQ=; b=b0e39IOEhwHxmJFd281W9qhgd6VTYgw/24AKGHhBUtbUxhXi58708gTIqrBDxMfC1E 2mnkO+r2ipxXMfHG9MVKmALtom9tWJzl3itDgAYZ8YqlopzBFN44xzhcG4phw2lvjbvs hnDWQhFaxokNmRPKD35FLhIsj8oO5fAmpwfjG0jqDhx/H7nFXSTis8BpX3wNlXtrmVW/ cODG04SAyBr3ksg8MABxiYuCdesRITxFFsIEUht71g8vKUwX7DnDLoPDsnDQzdslulK4 l4Bz56yP399ZAPplhcYgdxEJQnrTziuglfscXaZieEsuO9KDG6I3D+JTJ7iE0XdYXs9A ZhHg== Received: by 10.50.45.162 with SMTP id o2mr2542935igm.0.1346360774546; Thu, 30 Aug 2012 14:06:14 -0700 (PDT) X-Forwarded-To: linaro-patchwork@canonical.com X-Forwarded-For: patch@linaro.org linaro-patchwork@canonical.com Delivered-To: patches@linaro.org Received: by 10.50.184.232 with SMTP id ex8csp31367igc; Thu, 30 Aug 2012 14:06:13 -0700 (PDT) Received: by 10.216.24.140 with SMTP id x12mr3836400wex.101.1346360773275; Thu, 30 Aug 2012 14:06:13 -0700 (PDT) Received: from indium.canonical.com (indium.canonical.com. [91.189.90.7]) by mx.google.com with ESMTPS id 75si3581429wet.129.2012.08.30.14.06.12 (version=TLSv1/SSLv3 cipher=OTHER); Thu, 30 Aug 2012 14:06:13 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of bounces@canonical.com designates 91.189.90.7 as permitted sender) client-ip=91.189.90.7; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of bounces@canonical.com designates 91.189.90.7 as permitted sender) smtp.mail=bounces@canonical.com Received: from ackee.canonical.com ([91.189.89.26]) by indium.canonical.com with esmtp (Exim 4.71 #1 (Debian)) id 1T7BwK-0005ox-OR for ; Thu, 30 Aug 2012 21:06:12 +0000 Received: from ackee.canonical.com (localhost [127.0.0.1]) by ackee.canonical.com (Postfix) with ESMTP id A69A9E4B76 for ; Thu, 30 Aug 2012 21:06:12 +0000 (UTC) MIME-Version: 1.0 X-Launchpad-Project: lava-dispatcher X-Launchpad-Branch: ~linaro-validation/lava-dispatcher/trunk X-Launchpad-Message-Rationale: Subscriber X-Launchpad-Branch-Revision-Number: 378 X-Launchpad-Notification-Type: branch-revision To: Linaro Patch Tracker From: noreply@launchpad.net Subject: [Branch ~linaro-validation/lava-dispatcher/trunk] Rev 378: do not put the LAVA access token into the log file Message-Id: <20120830210612.21316.82585.launchpad@ackee.canonical.com> Date: Thu, 30 Aug 2012 21:06:12 -0000 Reply-To: noreply@launchpad.net Sender: bounces@canonical.com Errors-To: bounces@canonical.com Precedence: bulk X-Generated-By: Launchpad (canonical.com); Revision="15877"; Instance="launchpad-lazr.conf" X-Launchpad-Hash: 3c4aec6a6ab5970dc53becb89bdfd6ca300c39df X-Gm-Message-State: ALoCoQkRqJ8nrqDHQ0OfdVOjl35Wtfm2G1lC5itijUdAi2nBzKtjIDJLN32UVnsWG6x2kRju0QtK Merge authors: Michael Hudson-Doyle (mwhudson) Related merge proposals: https://code.launchpad.net/~mwhudson/lava-dispatcher/dont-show-token-bug-1042138/+merge/121983 proposed by: Michael Hudson-Doyle (mwhudson) review: Approve - Andy Doan (doanac) ------------------------------------------------------------ revno: 378 [merge] committer: Michael Hudson-Doyle branch nick: trunk timestamp: Fri 2012-08-31 09:05:07 +1200 message: do not put the LAVA access token into the log file modified: lava_dispatcher/job.py --- lp:lava-dispatcher https://code.launchpad.net/~linaro-validation/lava-dispatcher/trunk You are subscribed to branch lp:lava-dispatcher. To unsubscribe from this branch go to https://code.launchpad.net/~linaro-validation/lava-dispatcher/trunk/+edit-subscription === modified file 'lava_dispatcher/job.py' --- lava_dispatcher/job.py 2012-06-13 07:35:23 +0000 +++ lava_dispatcher/job.py 2012-08-30 04:22:50 +0000 @@ -228,8 +228,11 @@ params = submit_results.get('parameters', {}) action = lava_commands[submit_results['command']]( self.context) + params_for_display = params.copy() + if 'token' in params_for_display: + params_for_display['token'] = '' try: - logging.info("Submitting the test result with parameters = %s", params) + logging.info("Submitting the test result with parameters = %s", params_for_display) action.run(**params) except Exception as err: logging.error("Failed to submit the test result. Error = %s", err)