Code review comment for lp:~jcsackett/launchpad/anonymous-api-access-emails-681815

Revision history for this message
j.c.sackett (jcsackett) wrote :

I've pushed up a change which makes this a better test, but it's still going to pass in either event; oddly, while in test mode, this security hole doesn't seem to exist.

I encourage any thoughts as to how to deal with this; otherwise, I think the tests show this patch hasn't broken anything, and the scripts attached to the bug can be used to QA it once it's live on qastaging.

« Back to merge proposal