Page MenuHomePhabricator

Fix logged-out Diffusion calls to Conduit
ClosedPublic

Authored by epriestley on Oct 22 2013, 8:12 PM.
Tags
None
Referenced Files
F18958537: D7380.id16622.diff
Nov 13 2025, 4:47 AM
F18958087: D7380.diff
Nov 13 2025, 3:51 AM
F18923006: D7380.diff
Nov 9 2025, 4:10 PM
F18906830: D7380.id16619.diff
Nov 8 2025, 8:35 PM
F18824812: D7380.id16622.diff
Oct 23 2025, 6:01 PM
F18770845: D7380.diff
Oct 8 2025, 3:30 PM
F18616776: D7380.id16622.diff
Sep 14 2025, 6:57 PM
F18179797: D7380.id16619.diff
Aug 16 2025, 4:19 AM
Subscribers

Details

Reviewers
btrahan
Commits
Restricted Diffusion Commit
rP90b83d7a92c5: Fix logged-out Diffusion calls to Conduit
Summary

Conduit doesn't currently have an analog to "shouldAllowPublic", so the recent policy checks added here caught legitimate Conduit calls when viewing Diffusion as a logged-out user.

Add shouldAllowPublic() and set it for all the Diffusion queries.

(More calls probably need this, but we can add it when we hit them.)

Test Plan

Looked at Diffusion as a logged-out user with public access enabled.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped