> Just a note - in exception handling context (in "except:" suite), you can use
> log.exception("message") to make it log message and associated exception stack
> trace. Unless you of course want to suppress stack trace logging, then using
> log.error() is the way to go. I don't think the distinction is critical here,
> so looks ok, thanks.
I didn't about log.exception, which is why I used log.error instead. We can change it later if necessary.
> Just a note - in exception handling context (in "except:" suite), you can use "message" ) to make it log message and associated exception stack
> log.exception(
> trace. Unless you of course want to suppress stack trace logging, then using
> log.error() is the way to go. I don't think the distinction is critical here,
> so looks ok, thanks.
I didn't about log.exception, which is why I used log.error instead. We can change it later if necessary.