Launchpad itself

Code review comment for lp:~cjwatson/launchpad/librarian-accept-macaroon

  1. librarian-accept-macaroon
  2. Merge into devel
Revision history for this message
William Grant (wgrant) wrote :

In fact, the authserver interface is completely ambiguous. It so happens that CodeImportJobMacaroonIssuer.verifyMacaroon will crash if it's given an LFA ID, so it's safe for now, but a giant boobytrap.

review: Needs Fixing

« Back to merge proposal