Glance

Code review comment for lp:~rconradharris/glance/cached-images-middleware

  1. cached-images-middleware
  2. Merge into trunk
Revision history for this message
Jay Pipes (jaypipes) wrote :

> > Looking around, I see a few other places where we do `str(e)` as well.
>
> Heh, this isn't the first time I've made a comment like this about OS code. :)

FYI, I told Rick we can do a separate refactoring patch around this. I've logged a bug about it for tracking purposes: https://bugs.launchpad.net/glance/+bug/816077

-jay

« Back to merge proposal