Force .perform_logging to bool

[topiaruss]

Without this change, perform_logging ends up with the the value of any passed zipkin_attrs.

[coveralls]

Coverage remained the same at 100.0% when pulling 34fe76f on topiaruss:patch-1 into 7937ca8 on Yelp:master.

[coveralls]

Coverage remained the same at 100.0% when pulling d68957f on topiaruss:patch-1 into 7937ca8 on Yelp:master.

[coveralls]

Coverage remained the same at 100.0% when pulling d68957f on topiaruss:patch-1 into 7937ca8 on Yelp:master.

[coveralls]

Coverage remained the same at 100.0% when pulling d68957f on topiaruss:patch-1 into 7937ca8 on Yelp:master.

[coveralls]

Coverage remained the same at 100.0% when pulling d68957f on topiaruss:patch-1 into 7937ca8 on Yelp:master.

[coveralls]

Coverage remained the same at 100.0% when pulling 066d141 on topiaruss:patch-1 into 7937ca8 on Yelp:master.

[coveralls]

Coverage remained the same at 100.0% when pulling 066d141 on topiaruss:patch-1 into 7937ca8 on Yelp:master.

[coveralls]

Coverage remained the same at 100.0% when pulling 066d141 on topiaruss:patch-1 into 7937ca8 on Yelp:master.

[coveralls]

Coverage remained the same at 100.0% when pulling 066d141 on topiaruss:patch-1 into 7937ca8 on Yelp:master.

[coveralls]

Coverage remained the same at 100.0% when pulling ccecc6c on topiaruss:patch-1 into 7937ca8 on Yelp:master.

[coveralls]

Coverage remained the same at 100.0% when pulling ccecc6c on topiaruss:patch-1 into 7937ca8 on Yelp:master.

[coveralls]

Coverage remained the same at 100.0% when pulling ccecc6c on topiaruss:patch-1 into 7937ca8 on Yelp:master.

[coveralls]

Coverage remained the same at 100.0% when pulling ccecc6c on topiaruss:patch-1 into 7937ca8 on Yelp:master.

[kaisen]

Thanks for the PR! I'm probably going to merge this, but I'm curious: is this fixing a bug? From what I see, self.perform_logging is only evaluated for truthiness, so the code should work regardless of the actual type of self.perform_logging.

[topiaruss]

I think you are probably right. And I'm not aware of a bug.
I was debugging an issue and surprised to see a list in what looked like a boolean field.
Just cleaning it up, in passing.

[coveralls]

Coverage remained the same at 100.0% when pulling ccecc6c on topiaruss:patch-1 into 7937ca8 on Yelp:master.

[coveralls]

Coverage remained the same at 100.0% when pulling ccecc6c on topiaruss:patch-1 into 7937ca8 on Yelp:master.

[coveralls]

Coverage remained the same at 100.0% when pulling ccecc6c on topiaruss:patch-1 into 7937ca8 on Yelp:master.

[coveralls]

Coverage remained the same at 100.0% when pulling ccecc6c on topiaruss:patch-1 into 7937ca8 on Yelp:master.

[coveralls]

Coverage remained the same at 100.0% when pulling ccecc6c on topiaruss:patch-1 into 7937ca8 on Yelp:master.

[kaisen]

Merging anyway. Explicit is good :)

[topiaruss]

Well done. I'm enjoying using it. Thanks!