Closed Bug 821255 Opened 13 years ago Closed 13 years ago

Can't subscribe to developer newsletter

Categories

(developer.mozilla.org Graveyard :: Wiki pages, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mozilla, Unassigned)

References

Details

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:17.0) Gecko/20100101 Firefox/17.0 Build ID: 20121128204232 Steps to reproduce: Tried to sign up for the developer newsletter (Firefox OS/marketplace) Actual results: Tumbeasts (I think 501?) Expected results: Successfully subscribed to newsletter
This is sporadic. Error message is: BasketException: Error connecting to https://basket.mozilla.com/news/subscribe/: HTTP Error 500: INTERNAL SERVER ERROR. Ensure that BASKET_URL is configured correctly in your settings file. cc'ing pmac and mkelly: what's the tactic on handling these errors? Should we implement a queue for them? i.e., does this bug depend on bug 806327? We need to fix this up before MWC since I'm sure we'll promote the app developer newsletter there.
Nothing special here to say about design. Feel free to pull into "Technical research".
I'd say you can implement a queue for them if you already have a system in place. Bug 806327 is in the works and will implement a celery queue for the basket service, so it should take care of the issue if the problem is timeouts. We've experienced these a LOT on bedrock, and moving the subscribe calls to be async should solve a lot of issues. I can't find where in the basket service or client code it'd produce the error message you're seeing. Since the problem is sporadic, I strongly suspect the timeout issue.
Depends on: 806327
Just did a live test and it worked for me, so I can confirm it must be transient.
Les, Luke: Any chance we could get this fixed today? Need to have it ready in time for MWC.
Commits pushed to master at https://github.com/mozilla/kuma https://github.com/mozilla/kuma/commit/228ac82f6368b219b71e604076f4ba37008858d1 fix bug 821255 - retry basket for transient errors https://github.com/mozilla/kuma/commit/a91cc2909187c39c9b789436f375fe2749de1ac8 Merge pull request #882 from groovecoder/newsletter-exceptions-821255 fix bug 821255 - retry basket for transient errors
Status: UNCONFIRMED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Product: developer.mozilla.org → developer.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.