You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@drill.apache.org by cg...@apache.org on 2021/09/02 15:50:00 UTC

[drill] branch master updated: [MINOR-UPDATE]: Removed password from debug logging (#2305)

This is an automated email from the ASF dual-hosted git repository.

cgivre pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/drill.git


The following commit(s) were added to refs/heads/master by this push:
     new ac8b234  [MINOR-UPDATE]: Removed password from debug logging (#2305)
ac8b234 is described below

commit ac8b2343e5ac57b5b8cf4836d63d60dc7cf8b223
Author: Charles S. Givre <cg...@apache.org>
AuthorDate: Thu Sep 2 11:49:50 2021 -0400

    [MINOR-UPDATE]: Removed password from debug logging (#2305)
---
 tools/drill-patch-review.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/tools/drill-patch-review.py b/tools/drill-patch-review.py
index 131db04..0b4a857 100755
--- a/tools/drill-patch-review.py
+++ b/tools/drill-patch-review.py
@@ -65,6 +65,7 @@ def main():
   p.close()
 
   rb_command="post-review --publish --tracking-branch " + opt.branch + " --target-groups=drill-git --bugs-closed=" + opt.jira
+  rb_command_for_debug=rb_command + " --username " + opt.reviewboard_user + " --password XXXX"
   rb_command=rb_command + " --username " + opt.reviewboard_user + " --password " + opt.reviewboard_password
 
   if opt.debug:
@@ -80,7 +81,7 @@ def main():
   if opt.testing:
     rb_command=rb_command + " --testing-done=" + opt.testing
   if opt.debug:
-    print(rb_command)
+    print(rb_command_for_debug)
   p=os.popen(rb_command)
   rb_url=""
   for line in p: