tbutils ExceptionInfo and TracebackInfo to_dict() result not JSON serializable about boltons HOT 3 CLOSED

#114 is merged and a release is forthcoming. Thank you both for the report and the great fixes!

from boltons.

I can see two potential solutions to this @doroncohen:

The first is to make the resulting dictionary contain string instances of _DeferredLine. After all, the docstring for to_dict says "Get a :class:dict copy of the Callpoint. Useful for serialization."

This would effectively change the API though; I'm not sure what @mahmoud feels in regards to API stability here. There is always a slim chance this could break things for someone who is using to_dict in an unconventional or unexpected way.

This leaves another option: Provide a to_json method for all Callpoint object instances. This would result in a stable API and JSON serialization support. Potentially handy.

I actually lean towards the first solution personally because it seems weird to have a class designed for serialization that can't actually be easily serialized.

To that end I've created two pull requests, each implementing one version of the fix.

#113

and

#114

Really I'm not in a position to make the call here, but I figured it was easy enough to provide both potential solutions and let the boltons maintainers make a call.

from boltons.

Yeah, I'm probably the most extensive user of this API, but having gone through all my uses, they all stringify the DeferredLine, so not only won't the first solution break my code, but it probably should have been built-in to begin with. I'm going to do some more testing before release, but for now I think I'm going to go with #114.

from boltons.