Homec4science

Remove "feed.public" and sort out feed policies

Authored by epriestley <git@epriestley.com> on Jul 2 2015, 00:06.

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

Committed
epriestley <git@epriestley.com>Jul 2 2015, 00:06
Pushed
aubortJan 31 2017, 17:16
Parents
rPH7f67bd8aff02: Fix some pht-strings
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH6a996d76fd1a: Remove "feed.public" and sort out feed policies (authored by epriestley <git@epriestley.com>).Jul 2 2015, 00:06