Arcana / dotabank-web

dotabank.com
8 stars 6 forks source link

current_user being set to None #4

Closed rjackson closed 10 years ago

rjackson commented 10 years ago

https://app.getsentry.com/arcanaio/dotabank/group/23375109/

current_user should always be either an instance of AnonymousUser or an actual User object. I guess this occured because of the other load_user() failure (see https://app.getsentry.com/arcanaio/dotabank/group/23375105/ - seems to have been an anomaly from the WebAPI).

Solution is probably catching steamodd exceptions in User._update_steam_name, so a failure there doesn't bring the whole app down.