Bug 64165 - Rollout patches created by sheriffbot say that sheriffbot is the patch author, but should say that some contributor is the author
Summary: Rollout patches created by sheriffbot say that sheriffbot is the patch author...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-07-08 07:07 PDT by Adam Roben (:aroben)
Modified: 2011-07-08 08:40 PDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Roben (:aroben) 2011-07-08 07:07:49 PDT
When you ask sheriffbot to create a rollout patch (via "sheriffbot rollout"), sheriffbot itself is used as the patch author in the ChangeLog. Since sheriffbot is really just a proxy for the requester, it would make more sense if the requester were used as the author.
Comment 1 Adam Barth 2011-07-08 08:37:58 PDT
Historically, we've had the perspective that your IRC is completely untrusted because spoofing identity in IRC is very easy.  Another option is to have the person who marks the patch cq+ be the patch author.
Comment 2 Adam Barth 2011-07-08 08:38:16 PDT
*your IRC identity
Comment 3 Adam Roben (:aroben) 2011-07-08 08:40:55 PDT
(In reply to comment #1)
> Historically, we've had the perspective that your IRC is completely untrusted because spoofing identity in IRC is very easy.

Makes sense.

> Another option is to have the person who marks the patch cq+ be the patch author.

Maybe that's the best option.