archive/pillar
0
0
Fork 0
Code Issues 42 Pull Requests Projects Releases Wiki Activity

More logging in PATCH handler

Browse Source
This commit is contained in:
Sybren A. Stüvel
2017-05-18 15:30:58 +02:00
parent cbe182a298
commit c12b646b09
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
pillar/api/patch_handler.py
View File

@@ -47,8 +47,8 @@ class AbstractPatchHandler:
} }
if self.log.isEnabledFor(logging.INFO): if self.log.isEnabledFor(logging.INFO):
self.log.info('Creating PATCH handler %s%s for operations: %s', self.log.info('Creating PATCH handler %s.%s%s for operations: %s',
blueprint.name, self.route, blueprint.name, self.patch.__name__, self.route,
sorted(self.patch_handlers.keys())) sorted(self.patch_handlers.keys()))
blueprint.add_url_rule(self.route, blueprint.add_url_rule(self.route,
@@ -66,11 +66,13 @@ class AbstractPatchHandler:
real_object_id = str2id(object_id) real_object_id = str2id(object_id)
patch = request.get_json() patch = request.get_json()
if not patch: if not patch:
self.log.info('Bad PATCH request, did not contain JSON')
raise wz_exceptions.BadRequest('Patch must contain JSON') raise wz_exceptions.BadRequest('Patch must contain JSON')
try: try:
patch_op = patch['op'] patch_op = patch['op']
except KeyError: except KeyError:
self.log.info("Bad PATCH request, did not contain 'op' key")
raise wz_exceptions.BadRequest("PATCH should contain 'op' key to denote operation.") raise wz_exceptions.BadRequest("PATCH should contain 'op' key to denote operation.")
log.debug('User %s wants to PATCH "%s" %s %s', log.debug('User %s wants to PATCH "%s" %s %s',