[MERGE] dont stacktrace on bad plugin versions

Marius Kruger amanic at gmail.com
Thu Oct 16 01:06:56 BST 2008


2008/10/16 Forest Bond <forest at alittletooquiet.net>

> ...



> > +        try:
> > +            version_string = _format_version_tuple(version_info)
> > +        except:
> > +            # try to return something usefull for bad plugins, in stead
> of
> > +            # stack tracing.
> > +            version_string = '.'.join(map(str, version_info))
>
> This will mask other exceptions, including KeyboardInterrupt.  Should you
> specify which exceptions you may be expecting?


Good point, I suppose I can ignore that one.
I found code in server.py like:

       try:
            ...
        except KeyboardInterrupt:
            raise
        except Exception, e:
            trace.log_exception_quietly()

will that be ok?
is there any others we should ignore?

regards
marius
-------------- next part --------------
An HTML attachment was scrubbed...
URL: https://lists.ubuntu.com/archives/bazaar/attachments/20081016/7c9e7ab0/attachment.htm 


More information about the bazaar mailing list