Merge lp://staging/~adeuring/launchpad/bug-550973 into lp://staging/launchpad

Proposed by Abel Deuring
Status: Merged
Approved by: Deryck Hodge
Approved revision: no longer in the source branch.
Merged at revision: not available
Proposed branch: lp://staging/~adeuring/launchpad/bug-550973
Merge into: lp://staging/launchpad
Diff against target: 16 lines (+6/-0)
1 file modified
lib/canonical/launchpad/webapp/ (+6/-0)
To merge this branch: bzr merge lp://staging/~adeuring/launchpad/bug-550973
Reviewer Review Type Date Requested Status
Gary Poster (community) Approve
Deryck Hodge (community) code Approve
Review via email:

Description of the change

This branch just adds a comment why we don't require a referer header for POSTs to /+hwdb/+submit. See bug 550973

To post a comment you must log in.
Revision history for this message
Deryck Hodge (deryck) wrote :

I hope Gary won't mind, but I want to step in and review here. I believe I understand what he's looking for in terms of a comment, and we need to move this branch forward on our kanban board.

With that in mind, this looks good to me and I'm marking it approved.

review: Approve (code)
Revision history for this message
Gary Poster (gary) wrote :

Looks good, and sorry for not getting to it sooner.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.