DescriptionRemove force push from GitCheckout.commit()
The CQ should not be able to force push -- this allows it to wipe out legitimate
commits if something goes wrong internally. This happened recently, with a CQ
commit orphaning 20 commits from the previous few days. This may have happened
any number of times previously, probably only wiping out one commit at a time,
whenever there was a race between the CQ and a developer's direct commit+push.
Repositories should also protect themselves by disallowing force push in their
ACLs.
BUG=261619
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=258092
Patch Set 1 #
Messages
Total messages: 8 (0 generated)
|