launchpad-reviewers team mailing list archive

Thread
Date

Re: [Merge] lp:~deryck/launchpad/404-bug-api-generates-oops-701246 into lp:launchpad

To: Deryck Hodge <deryck.hodge@xxxxxxxxxxxxx>
From: Robert Collins <robertc@xxxxxxxxxxxxxxxxx>
Date: Wed, 18 May 2011 18:52:42 -0000
In-reply-to: <20110518135110.25573.26984.launchpad@loganberry.canonical.com>
Reply-to: mp+61407@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Hah, so we want the referer for 404s from locally generated URLs to detect missing links. I don't think we need them to be OOPSes though.

OTOH as we have our report based on OOPSes today it may be nice to do the referrer check in the code you're changing.
-- 
https://code.launchpad.net/~deryck/launchpad/404-bug-api-generates-oops-701246/+merge/61407
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~deryck/launchpad/404-bug-api-generates-oops-701246 into lp:launchpad.

References

[Merge] lp:~deryck/launchpad/404-bug-api-generates-oops-701246 into lp:launchpad
From: Deryck Hodge, 2011-05-18