diff options
author | Andrew Laski <andrew@lascii.com> | 2016-07-01 13:51:48 -0400 |
---|---|---|
committer | Andrew Laski <andrew@lascii.com> | 2016-07-01 14:16:06 -0400 |
commit | 219d970d130613b9723381990825b05e104bfe12 (patch) | |
tree | 4a48ce7ea07adb9d0438b05a5df9b2d215e20d75 /HACKING.rst | |
parent | 7ee75f638af7f17285508b10ac49346a51677714 (diff) | |
download | nova-219d970d130613b9723381990825b05e104bfe12.tar.gz |
Hacking check for _ENFORCER.enforce()
In order to ensure that only registered policies are used for
authorization checks _ENFORCER.authorize should be used rather than
_ENFORCER.enforce. This adds a check to look for instances of
_ENFORCER.enforce being used.
Change-Id: Iee78e93a3e1d4c6c30681b18698b7fc9cb6fa982
Implements: bp policy-in-code
Diffstat (limited to 'HACKING.rst')
-rw-r--r-- | HACKING.rst | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/HACKING.rst b/HACKING.rst index 632e9ad0e0..721326db4e 100644 --- a/HACKING.rst +++ b/HACKING.rst @@ -61,6 +61,7 @@ Nova Specific Commandments - [N348] Deprecated library function os.popen() - [N349] Check for closures in tests which are not used - [N350] Policy registration should be in the central location ``nova/policies/`` +- [N351] Do not use the oslo_policy.policy.Enforcer.enforce() method. Creating Unit Tests ------------------- |