Code review comment for lp://staging/~gary/launchpad/bug553368

Revision history for this message
Robert Collins (lifeless) wrote :

We probably want to log the OOPS, but show a nice error page.

Why log the OOPS? because our SSO should - never - be down, so if its down or misbehaving, we want to respond promptly to that.

What do think?

The code itself looks fine, modulo this conceptual question.

review: Needs Information

« Back to merge proposal