Page MenuHomePhabricator

Simplify custom policies before saving, and reject meaningless policies
ClosedPublic

Authored by epriestley on Oct 14 2013, 11:10 PM.
Tags
None
Referenced Files
F18627768: D7314.diff
Tue, Sep 16, 4:44 AM
F18625263: D7314.id16474.diff
Mon, Sep 15, 10:20 PM
F18620956: D7314.id16479.diff
Mon, Sep 15, 6:45 AM
F18616211: D7314.id16474.diff
Sun, Sep 14, 4:36 PM
F18584259: D7314.id16479.diff
Thu, Sep 11, 12:26 PM
F18185057: D7314.id16475.diff
Aug 16 2025, 2:58 PM
F17987112: D7314.id16479.diff
Aug 1 2025, 10:18 PM
F17946389: D7314.id.diff
Jul 31 2025, 2:21 PM
Subscribers

Details

Summary

Ref T603. Do a little more sanity checking on custom policies, so policies like this:

[ Allow ] [ Users ] [ <no users> ]

...that don't specify anything and thus which aren't meaningful raise errors.

Test Plan

{F69570}

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley updated this revision to Unknown Object (????).Oct 14 2013, 11:12 PM
  • Save the simplified version of the data.