<logging.Logger instance at 0x927d3ec> is not JSON serializable

CityGenerator / Megacosm-Generator

A tool for creating fantasy campaign settings, including maps, regions, flags, cities, NPCs, businesses, legends, lore and more.

GNU General Public License v2.0

69 stars 30 forks source link

<logging.Logger instance at 0x927d3ec> is not JSON serializable #101

Closed mareofnight closed 10 years ago

mareofnight commented 10 years ago

URL: http://megacosm.morgajel.net/flag seed: 9709265 time: 2014-04-18 09:48:19.494294

(Do you want me to look into this? I've got a working version of flag elsewhere, so I suspect it'll be fixed when my refactor is merged in anyway.)

morgajel commented 10 years ago

ooops! let me look into that; I need to do some testing around it to make sure I catch it.

morgajel commented 10 years ago

all better now; I added a new logger object and forgot to exclude it from flags. Since /flag didn't have a test, I didn't get an error that it was throwing a 500. New test in place for that, and logger removed before jsonifying.

morgajel commented 10 years ago

Also, you can see how simple this was to fix by checking out e2f077f

mareofnight commented 10 years ago

Nice ^^