Page MenuHomePhabricator
Differential D21567

Apply "pcre.*_limit" ini options in web environments
ClosedPublic
Actions

Authored by epriestley on Feb 18 2021, 7:21 PM.
Tags
None
Referenced Files
F13087243: D21567.diff
Thu, Apr 25, 12:55 AM
F13081350: D21567.id.diff
Wed, Apr 24, 5:59 PM
Unknown Object (File)
Sat, Apr 20, 5:34 PM
Unknown Object (File)
Sat, Apr 20, 4:44 AM
Unknown Object (File)
Thu, Apr 11, 8:40 AM
Unknown Object (File)
Wed, Mar 27, 8:18 PM
Unknown Object (File)
Wed, Mar 27, 8:17 PM
Unknown Object (File)
Wed, Mar 27, 8:17 PM
View All 74 Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13100: PCRE segfaults readily with default "pcre.backtrack_limit" and "pcre.recursion_limit" values
T13586: In Herald transcripts, surface exceptions encountered while evaluating fields
Commits
rP20a54a3006de: Apply "pcre.*_limit" ini options in web environments
Summary

Ref T13100. Ref T13586. See D21566, which applied these changes to CLI environments. Also apply them to web environments.

Test Plan

Loaded Phabricator.

Diff Detail

Repository
rP Phabricator
Branch
herald6
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 25147
Build 34707: Run Core Tests
Build 34706: arc lint + arc unit

Event Timeline

epriestley created this revision.Feb 18 2021, 7:21 PM
Harbormaster completed remote builds in B25147: Diff 51338.Feb 18 2021, 7:23 PM
epriestley requested review of this revision.Feb 18 2021, 7:23 PM
This revision was not accepted when it landed; it landed in state Needs Review.Feb 19 2021, 7:16 PM
Closed by commit rP20a54a3006de: Apply "pcre.*_limit" ini options in web environments (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rP20a54a3006de: Apply "pcre.*_limit" ini options in web environments.

Revision Contents
Changeset List

PathSize
support/
startup/
18 lines

Diff 51338

View Options

support/startup/PhabricatorStartup.php

Loading...
Phabricator · Built by Phacility