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)
developer.mozilla.org Graveyard
Wiki pages
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
Comment 1•13 years ago
|
||
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.
Comment 2•13 years ago
|
||
Nothing special here to say about design. Feel free to pull into "Technical research".
Comment 3•13 years ago
|
||
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
Comment 4•13 years ago
|
||
Just did a live test and it worked for me, so I can confirm it must be transient.
Comment 5•13 years ago
|
||
Les, Luke: Any chance we could get this fixed today? Need to have it ready in time for MWC.
Comment 6•13 years ago
|
||
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
Updated•13 years ago
|
Status: UNCONFIRMED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Product: developer.mozilla.org → developer.mozilla.org Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•