From a229251f622c2cdddfd546e7bf00fb9b34b08596 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sybren=20A=2E=20St=C3=BCvel?= Date: Mon, 9 May 2016 14:34:25 +0200 Subject: [PATCH] Gracefully handle missing 'allowed_methods' --- pillarsdk/projects.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/pillarsdk/projects.py b/pillarsdk/projects.py index 5bb3d63..68168c3 100644 --- a/pillarsdk/projects.py +++ b/pillarsdk/projects.py @@ -96,9 +96,9 @@ class Project(List, Find, Create, Post, Update, Delete, Replace): node_type = next((item for item in response['node_types'] if item['name'] and item['name'] == node_type_name), None) - if method in node_type['allowed_methods']: - return True - return False + if node_type is None or 'allowed_methods' not in node_type: + return False + return method in node_type['allowed_methods'] def _manage_user(self, user_id, action, api=None): """Add or remove a user to a project give its ObjectId."""