Page MenuHomePhabricator

Audit code does not fully respect committer identities as commit authors
Closed, ResolvedPublic

Description

See PHI1303. In this issue, a user is the author of a commit only via repository identity. Audit is confused about this and is treating the commit as though it has no author.

This is probably a getAuthorPHID() somewhere that should be a getCommitIdentityAuthorPHID().

Likely reproduction steps are:

  • Commit and push as quack <xyz@abc.example> (a made-up identity string with no corresponding user).
  • Manually set the user associated with this identity to alice (or whatever).
  • View the commit as alice.

Expected behavior:

  • Alice has audit actions as the commit author.

Anticipated behavior:

  • Alice has auditor actions as though she were a non-author.

Revisions and Commits

Restricted Differential Revision
Restricted Differential Revision

Event Timeline

epriestley triaged this task as Normal priority.Jun 14 2019, 4:02 PM
epriestley created this task.
epriestley added a revision: Restricted Differential Revision.Jun 17 2019, 5:04 PM
epriestley added a revision: Restricted Differential Revision.Jun 17 2019, 5:11 PM
epriestley added a commit: Restricted Diffusion Commit.Jun 17 2019, 8:47 PM
epriestley closed this task as Resolved by committing Restricted Diffusion Commit.Jun 17 2019, 8:56 PM
epriestley claimed this task.
epriestley added a commit: Restricted Diffusion Commit.