diff options
author | cpettet <rush@wikimedia.org> | 2014-11-19 22:20:58 -0600 |
---|---|---|
committer | cpettet <rush@wikimedia.org> | 2014-11-19 22:20:58 -0600 |
commit | 3e84e3075dcca0228f9c9f7ea48abf1447a7ea03 (patch) | |
tree | 863d8f4e53cf9a5ceaa754c7f0a43890d5705a20 | |
parent | f9ba5228d336d7106fddae36db3b88ce492f0934 (diff) |
don't mask security issues...in security states
-rwxr-xr-x | bugzilla_create.py | 2 | ||||
-rwxr-xr-x | wmfphablib/bzlib.py | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/bugzilla_create.py b/bugzilla_create.py index a797502..a57b013 100755 --- a/bugzilla_create.py +++ b/bugzilla_create.py @@ -172,7 +172,7 @@ def create(bugid): for c in com: if not isinstance(c, dict): c = ast.literal_eval(c) - clean_c = bzlib.build_comment(c) + clean_c = bzlib.build_comment(c, buginfo['secstate']) clean_com.append(clean_c) log('project: ' + buginfo['project']) diff --git a/wmfphablib/bzlib.py b/wmfphablib/bzlib.py index 6e1f8b0..92fc7ef 100755 --- a/wmfphablib/bzlib.py +++ b/wmfphablib/bzlib.py @@ -21,7 +21,7 @@ def sanitize_project_name(product, component): component_separator, component) -def build_comment(c): +def build_comment(c, secstate): """ takes a native bz comment dict and outputs a dict ready for processing into phab """ @@ -43,7 +43,7 @@ def build_comment(c): if c['count'] == 0: clean_c['bug_id'] = c['bug_id'] - if c['is_private']: + if c['is_private'] and secstate == 'none': c['text'] = security_mask attachment = find_attachment_in_comment(c['text']) |