Better error handling on Skill checking, also fixes EVE Proxt blowing auth params out the window

This commit is contained in:
2011-05-31 13:41:10 +01:00
parent c75a8fbc1f
commit c9f5599500
2 changed files with 13 additions and 10 deletions

View File

@@ -130,6 +130,7 @@ def import_eve_character_func(character_id, api_key=None, user_id=None, logger=l
logger.error('Error retrieving SkillInTraining.xml.aspx for User ID %s, Character ID %s - %s' % (user_id, character_id, exc)) logger.error('Error retrieving SkillInTraining.xml.aspx for User ID %s, Character ID %s - %s' % (user_id, character_id, exc))
else: else:
queuedoc = basic_xml_parse_doc(skillqueue) queuedoc = basic_xml_parse_doc(skillqueue)
if 'error' in queuedoc['eveapi'] and 'result' in queuedoc['eveapi']:
queuedoc = queuedoc['eveapi']['result'] queuedoc = queuedoc['eveapi']['result']
EVEPlayerCharacterSkill.objects.filter(character=pchar).update(in_training=0) EVEPlayerCharacterSkill.objects.filter(character=pchar).update(in_training=0)
if int(queuedoc['skillInTraining']): if int(queuedoc['skillInTraining']):

View File

@@ -100,9 +100,11 @@ class CachedDocumentManager(models.Manager):
except: except:
pass pass
else: else:
for k in ['userid', 'apikey', 'vcode', 'keyid']: fparams = {}
if k in params: del params[k] for k in params:
ApiAccessLog(userid=v, service='Unknown', time_access=doc.time_retrieved, document=self.construct_url(url_path, params)).save() if not k in ['userid', 'apikey', 'vcode', 'keyid']: fparams[k] = params[k]
ApiAccessLog(userid=v, service='Unknown', time_access=doc.time_retrieved, document=self.construct_url(url_path, fparams)).save()
return doc return doc