Home GnuPG

Explicitly decline to add commit authors as auditors from Herald
89d1403fe89dUnpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

Explicitly decline to add commit authors as auditors from Herald

Summary:
Fixes T12304. If you have a Herald rule which tries to add a commit author as an auditor, it fails validation when trying to apply.

Stop trying to apply these transactions, and explicitly tell the user why. Differential already uses a similar ruleset around reviewers, but Audit was using older code.

Test Plan:

  • Wrote a Herald rule to add A, B and C as auditors.
  • Committed as A.
  • After change, saw B and C added with transacript guidance that A was the author.

{F3235660}

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T12304

Differential Revision: https://secure.phabricator.com/D17404

Details

Provenance
epriestley <git@epriestley.com>Authored on Feb 23 2017, 11:42 PM
Parents
rPHAB3b6a651b69fe: Merge multiple Auditors transactions from Herald
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHAB89d1403fe89d: Explicitly decline to add commit authors as auditors from Herald (authored by epriestley <git@epriestley.com>).Feb 24 2017, 12:19 AM