Closed Bug 831390 Opened 11 years ago Closed 10 years ago

Add postback fail threshold to trigger re-reviews

Categories

(Marketplace Graveyard :: Reviewer Tools, defect, P2)

defect

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: robhudson, Unassigned)

References

Details

From bug 797927... "If a certain number of postbacks fail, they should probably go into a re-review queue."
Sounds like a good idea. If postbacks are failing then the app is taking money without providing the goods.
Priority: -- → P2
This bug seems like a good idea but we (the triagers) are a little confused about the specifics.  Would you write a few bullet points of what (or where in the code) we're counting?
Flags: needinfo?(kumar.mcmillan)
A postback fails and is counted/retried here: https://github.com/mozilla/webpay/blob/master/webpay/pay/utils.py#L67 It could fail for many reasons but if it fails repeatedly then the app server probably has a problem. After giving up, we could post to the Martketplace API to disable the app (or whatever).
Flags: needinfo?(kumar.mcmillan)
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.