Page MenuHomePhabricator

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

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
Subscribers
None

Details

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
Branch
preg1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19747
Build 26748: Run Core Tests
Build 26747: arc lint + arc unit

Event Timeline

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.
This revision was automatically updated to reflect the committed changes.