austgl / temars-eve-api

Automatically exported from code.google.com/p/temars-eve-api
0 stars 0 forks source link

Ignore errors during downtime #8

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
TEA takes error code 900 and above as a API failure and removes access to the 
user, TEA really should ignore any errors of code > 900 as they relate to 
either downtime or API server issues.

Version 1.1.1

What steps will reproduce the problem?
1. EVE API down
2. Logout/Login

List of codes from ErrorList.xml.aspx

  <row errorCode="901" errorText="Web site database temporarily disabled." /> 
  <row errorCode="902" errorText="EVE backend database temporarily disabled." /> 
  <row errorCode="903" errorText="Rate limited [{0}]: please obey all cachedUntil timers." /> 
  <row errorCode="999" errorText="User forced test error condition." /> 

Original issue reported on code.google.com by nikd...@gmail.com on 17 Aug 2010 at 12:12

GoogleCodeExporter commented 9 years ago
Line 281 and on in TEA.php, the check/continue for error code 900s needs to be 
above the DB query.

Original comment by nikd...@gmail.com on 17 Aug 2010 at 12:20

GoogleCodeExporter commented 9 years ago
it doesnt need to be above that, that is simple a status and doesnt affect 
actual access
i have changed it so it used return instead of continue so changed done lower 
down never happen

Original comment by tema...@googlemail.com on 14 Sep 2010 at 1:16