From ba5135076863046e3df1cf14698c39f2cf3af81a Mon Sep 17 00:00:00 2001 From: Ian Gulliver Date: Tue, 27 May 2014 14:32:15 -0700 Subject: [PATCH] Move exception logging to warn level, so real errors stick out. --- api.py | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/api.py b/api.py index 4724087..904de28 100644 --- a/api.py +++ b/api.py @@ -63,12 +63,12 @@ def Pin(google_user, client, instance_id, args): models.Subject.FindOrCreate(subject).Pin( message, client.parent_key(), sender_message_id, instance) except models.DuplicateMessage: - logging.exception('Duplicate message: %s', sender_message_id) + logging.warning('Duplicate pin: %s', sender_message_id) return { 'result': 'duplicate_message', } except models.AccessDenied: - logging.exception('Pin access denied') + logging.warning('Pin access denied') return { 'result': 'access_denied', } @@ -87,12 +87,12 @@ def SendMessage(google_user, client, instance_id, args): models.Subject.FindOrCreate(subject).SendMessage( message, client.parent_key(), sender_message_id) except models.DuplicateMessage: - logging.exception('Duplicate message: %s', sender_message_id) + logging.warning('Duplicate message: %s', sender_message_id) return { 'result': 'duplicate_message', } except models.AccessDenied: - logging.exception('SendMessage access denied') + logging.warning('SendMessage access denied') return { 'result': 'access_denied', } @@ -121,7 +121,7 @@ def Subscribe(google_user, client, instance_id, args): subject, client, instance, messages, last_id), } except models.AccessDenied: - logging.exception('Subscribe access denied') + logging.warning('Subscribe access denied') return { 'result': 'access_denied', } @@ -136,7 +136,7 @@ def Unpin(google_user, client, instance_id, args): models.Subject.FindOrCreate(subject).Unpin( client.parent_key(), sender_message_id, instance.key()) except models.AccessDenied: - logging.exception('Pin access denied') + logging.warning('Pin access denied') return { 'result': 'access_denied', }