FAForever / server

The servercode for the Forged Alliance Forever lobby

Home Page:http://www.faforever.com

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Error handling in configuration refresh loop

Askaholic opened this issue · comments

There should really be a try-catch around this block of code to make sure the configuration refresh loop doesn't crash:

self._logger.debug("Refreshing configuration variables")

Also this log statement should use %r for formatting with repr so you can tell the difference between strings and other types.

self._logger.info("New value for %s: %s -> %s", key, old_value, new_value)