Page MenuHomePhabricator
Differential D19178

Provide a more tailored error message when a Herald rule fails because of PCRE limits
ClosedPublic
Actions

Authored by epriestley on Mar 6 2018, 8:16 PM.
Tags
None
Referenced Files
F18788867: D19178.id.diff
Wed, Oct 15, 6:50 AM
F18742515: D19178.id.diff
Thu, Oct 2, 7:55 PM
F18736130: D19178.diff
Wed, Oct 1, 4:56 AM
F18509395: D19178.id.diff
Sep 5 2025, 3:23 AM
F18502327: D19178.diff
Sep 4 2025, 10:21 PM
F18234991: D19178.id.diff
Aug 20 2025, 6:57 PM
F18213463: D19178.diff
Aug 19 2025, 4:48 AM
F18094951: D19178.id.diff
Aug 7 2025, 10:26 PM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13100: PCRE segfaults readily with default "pcre.backtrack_limit" and "pcre.recursion_limit" values
Commits
rPb52783af80bd: (stable) Provide a more tailored error message when a Herald rule fails because…
rP573bf15124bb: Provide a more tailored error message when a Herald rule fails because of PCRE…
Summary

Ref T13100. Since rules may begin failing for PRCE configuration reasons soon, provide a more complete explanation of possible causes in the UI.

Test Plan

Faked this, hit it via test console, saw explanation in web UI.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Mar 6 2018, 8:16 PM
Harbormaster completed remote builds in B19747: Diff 45936.Mar 6 2018, 8:17 PM
This revision was not accepted when it landed; it landed in state Needs Review.Mar 6 2018, 8:17 PM
epriestley requested review of this revision.
Closed by commit rP573bf15124bb: Provide a more tailored error message when a Herald rule fails because of PCRE… (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
herald/
adapter/
8 lines

Diff 45938

View Options

src/applications/herald/adapter/HeraldAdapter.php

Loading...
Phabricator · Built by Phacility