Page MenuHomePhabricator

Lock feed.public and feed.http-hooks config options
ClosedPublic

Authored by epriestley on Dec 28 2014, 4:29 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 8:39 AM
Unknown Object (File)
Wed, Apr 10, 4:34 PM
Unknown Object (File)
Mar 13 2024, 4:57 AM
Unknown Object (File)
Feb 28 2024, 10:06 AM
Unknown Object (File)
Feb 28 2024, 9:57 AM
Unknown Object (File)
Feb 28 2024, 9:57 AM
Unknown Object (File)
Feb 28 2024, 9:55 AM
Unknown Object (File)
Feb 3 2024, 4:15 AM
Subscribers

Details

Summary

Ref T6817. Ref T5726. These both bypass policy checks, and would allow an attacker who gains control of an administrative account to enable public feed, then view feed stories they could not normally see; or enable feed.http-hooks, then read the posted text.

In the longer term I'd like to remove feed.public completely (possibly providing API alternatives, if necessary).

Test Plan

Looked at options in web UI and saw them locked.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad edited edge metadata.
This revision is now accepted and ready to land.Dec 29 2014, 3:03 AM
This revision was automatically updated to reflect the committed changes.