Add "raw author name" and "raw committer name" as Herald fields for commit content hooks
ClosedPublic
Actions

Authored by epriestley on Dec 27 2013, 12:04 AM.

Tags

None

Referenced Files

	F13138281: D7841.diff
	Thu, May 2, 11:51 PM

	Unknown Object (File)
	Sat, Apr 27, 12:23 PM

	Unknown Object (File)
	Sat, Apr 27, 11:13 AM

	Unknown Object (File)
	Wed, Apr 24, 5:26 PM

	Unknown Object (File)
	Mon, Apr 22, 10:38 PM

	Unknown Object (File)
	Sat, Apr 20, 6:14 PM

	Unknown Object (File)
	Fri, Apr 12, 12:25 AM

	Unknown Object (File)
	Thu, Apr 11, 7:31 AM

View All Files

Subscribers

aran

SEJeff

Details

Reviewers

btrahan

Maniphest Tasks

T4195: Implement pre-commit Herald rules

Commits

Restricted Diffusion Commit
rP9f38aaa5dec4: Add "raw author name" and "raw committer name" as Herald fields for commit…

Summary

Ref T4195. A legitimate rule which needs this field is "do not allow commits as root". Interestingly, we have exactly one commit as root in each Phabricator, Arcanist and libphutil.

Since the committer and author don't need to be Phabricator accounts (just the Pusher), the existing "Committer" and "Author" fields can't express this rule (they'll be empty).

Test Plan

{F93406}