nikdoof/test-auth
1
0
Fork 0
mirror of https://github.com/nikdoof/test-auth.git synced 2025-12-17 11:49:29 +00:00
Code Issues Packages Projects Releases Wiki Activity

Ignore common and handled error messages

Browse Source
This commit is contained in:
Andrew Williams
2011-09-28 12:15:54 +01:00
parent b4d65a3185
commit 13a3939889
1 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
app/eve_proxy/models.py
View File

@@ -14,6 +14,9 @@ API_URL = getattr(settings, 'EVE_API_URL', 'https://api.eve-online.com')
# Errors to rollback if we have a cached version of the document # Errors to rollback if we have a cached version of the document
ROLLBACK_ERRORS = range(516, 902) ROLLBACK_ERRORS = range(516, 902)
# Errors ignored if encountered, as they're valid responses in some cases
IGNORED_ERRORS = range(200, 223)
class CachedDocumentManager(models.Manager): class CachedDocumentManager(models.Manager):
""" """
This manager handles querying or retrieving CachedDocuments. This manager handles querying or retrieving CachedDocuments.
@@ -104,15 +107,18 @@ class CachedDocumentManager(models.Manager):
if error: if error:
# If we have a rollback error, try and retreive a correct version from the DB # If we have a rollback error, try and retreive a correct version from the DB
if error in ROLLBACK_ERRORS: if int(error) in ROLLBACK_ERRORS:
try: try:
doc = self.get(pk=doc.pk) doc = self.get(pk=doc.pk)
except self.DoesNotExist: except self.DoesNotExist:
doc.save() doc.save()
else: else:
if not int(error) in IGNORED_ERRORS:
logger.error("API Error %s encountered - %s" % (error, url), extra={'data': {'api-url': url, 'error': error, 'document': doc.body}}) logger.error("API Error %s encountered - %s" % (error, url), extra={'data': {'api-url': url, 'error': error, 'document': doc.body}})
else: else:
doc.save() doc.save()
else:
doc.save()
# If this is user related, write a log instance # If this is user related, write a log instance
if params and (params.get('userid', None) or params.get('keyid', None)): if params and (params.get('userid', None) or params.get('keyid', None)):