HomePhorge

Remove "feed.public" and sort out feed policies

Description

Remove "feed.public" and sort out feed policies

Summary:
Ref T6817. Fixes T8731. On the old secure host, feed.public was set to true. I didn't bring the option over, which caused the secondary issue in T8731.

Specifically, when feed.public is off, a logged-out user looking at feed can't see any stories, so they query all of feed until they hit the time limit.

To fix this immediately, just use the most open policy, which is basically equivalent but always correct.

To fix this more thoroughly:

  • Remove feed.public, which violates policies and has been slated for removal for a while (see T6817).
  • Clean up policy handling.

Test Plan:

  • As a logged-out user, viewed feed on a public install with feed.public off; no longer saw all stories get queried + no feed shown.
  • Grepped for feed.public.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: chad, epriestley

Maniphest Tasks: T6817, T8731

Differential Revision: https://secure.phabricator.com/D13518

Details

Provenance
epriestleyAuthored on Jul 1 2015, 3:06 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP7f67bd8aff02: Fix some pht-strings
Branches
Unknown
Tags
Unknown

Event Timeline