Page MenuHomePhabricator
Differential D19297

Fix a commit hook issue where pushing dangerous changes would fatal before hitting the dragon bureaucrats
ClosedPublic
Actions

Authored by epriestley on Apr 5 2018, 1:15 PM.
Tags
None
Referenced Files
F18509510: D19297.id.diff
Fri, Sep 5, 3:26 AM
F18502571: D19297.diff
Thu, Sep 4, 10:31 PM
F18443454: D19297.diff
Sun, Aug 31, 6:59 PM
F18095032: D19297.diff
Aug 7 2025, 10:30 PM
F18050806: D19297.id46180.diff
Aug 3 2025, 9:05 PM
F18050352: D19297.id46181.diff
Aug 3 2025, 7:27 PM
F18050351: D19297.id.diff
Aug 3 2025, 7:27 PM
F17994549: D19297.diff
Aug 2 2025, 3:18 AM
View All Files
Subscribers
None

Details

Reviewers
None
Commits
rP0fef964d86de: (stable) Fix a commit hook issue where pushing dangerous changes would fatal…
rP3de002c8414a: Fix a commit hook issue where pushing dangerous changes would fatal before…
Summary

See https://discourse.phabricator-community.org/t/php-fatal-when-using-git-push-d/1317. The behavioral changes for Herald on initial import from D19265 could leave $all_updates undefined if we throw early enough.

Test Plan

Pushed a dangerous change, saw dragon bureaucrats again.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Apr 5 2018, 1:15 PM
Harbormaster completed remote builds in B20001: Diff 46180.Apr 5 2018, 1:16 PM
epriestley requested review of this revision.Apr 5 2018, 1:16 PM
This revision was not accepted when it landed; it landed in state Needs Review.Apr 5 2018, 1:19 PM
Closed by commit rP3de002c8414a: Fix a commit hook issue where pushing dangerous changes would fatal before… (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
diffusion/
engine/
1 line

Diff 46181

View Options

src/applications/diffusion/engine/DiffusionCommitHookEngine.php

Loading...
Phabricator · Built by Phacility