Reviewed-by: Daniel Barboza <danielhb@linux.vnet.ibm.com> On 03/04/2014 07:01 AM, shaohef@linux.vnet.ibm.com wrote:

From: ShaoHe Feng <shaohef@linux.vnet.ibm.com>

Neither of these two removed InvalidOperation is used.

Signed-off-by: ShaoHe Feng <shaohef@linux.vnet.ibm.com> --- src/kimchi/auth.py | 2 -- 1 file changed, 2 deletions(-)

diff --git a/src/kimchi/auth.py b/src/kimchi/auth.py index f8ccea1..af3b610 100644 --- a/src/kimchi/auth.py +++ b/src/kimchi/auth.py @@ -197,13 +197,11 @@ def kimchiauth(admin_methods=None): debug("Entering kimchiauth...") if check_auth_session(): if not has_permission(admin_methods): - e = InvalidOperation('KCHAUTH0004E') raise cherrypy.HTTPError(403) return

if check_auth_httpba(): if not has_permission(admin_methods): - e = InvalidOperation('KCHAUTH0004E') raise cherrypy.HTTPError(403) return