finish masking out email addresses
Mask out email addresses of QA contacts and when adding components to BZ. Additionally, use descriptive placeholder if QA contact is unset and we default to the extras-qa alias. Signed-off-by: Nils Philippsen <nils@redhat.com>
This commit is contained in:
parent
78ac940933
commit
5211712f23
1 changed files with 18 additions and 2 deletions
|
@ -256,6 +256,7 @@ class BugzillaProxy:
|
|||
qacontact_email = self._get_bugzilla_email(qacontact)
|
||||
else:
|
||||
qacontact_email = 'extras-qa@fedoraproject.org'
|
||||
qacontact = f'<default: extras-qa@...>'
|
||||
|
||||
# Check for changes to the owner, qacontact, or description
|
||||
if product[pkg_key]['initialowner'] != owner_email:
|
||||
|
@ -284,9 +285,13 @@ class BugzillaProxy:
|
|||
print(f'[EDITCOMP] {data["product"]}/{data["component"]}')
|
||||
for key in ["initialowner", "description", "initialqacontact", "initialcclist"]:
|
||||
if data.get(key):
|
||||
if print_fas_names and key in ('initialowner', 'initialcclist'):
|
||||
if print_fas_names and key in ('initialowner',
|
||||
'initialqacontact',
|
||||
'initialcclist'):
|
||||
if key == 'initialowner':
|
||||
value = owner
|
||||
elif key == 'initialqacontact':
|
||||
value = qacontact
|
||||
else:
|
||||
value = initial_cc_fasnames
|
||||
print(f" {key} changed to FAS name(s) `{value}`")
|
||||
|
@ -313,6 +318,7 @@ class BugzillaProxy:
|
|||
qacontact_email = self._get_bugzilla_email(qacontact)
|
||||
else:
|
||||
qacontact_email = 'extras-qa@fedoraproject.org'
|
||||
qacontact = f'<default: extras-qa@...>'
|
||||
|
||||
data = {
|
||||
'product': self.config['products'][collection],
|
||||
|
@ -327,7 +333,17 @@ class BugzillaProxy:
|
|||
if self.config["verbose"]:
|
||||
print('[ADDCOMP] %s/%s' % (data["product"], data["component"]))
|
||||
for key in ["initialowner", "description", "initialqacontact", "initialcclist"]:
|
||||
if data.get(key):
|
||||
if print_fas_names and key in ('initialowner',
|
||||
'initialqacontact',
|
||||
'initialcclist'):
|
||||
if key == 'initialowner':
|
||||
value = owner
|
||||
elif key == 'initialqacontact':
|
||||
value = qacontact
|
||||
else:
|
||||
value = initial_cc_fasnames
|
||||
print(f" {key} set to FAS name(s) `{value}`")
|
||||
else:
|
||||
print(f" {key} set to {data.get(key)}")
|
||||
if not self.config["dryrun"]:
|
||||
try:
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue