Closed Bug 1331604 Opened 6 years ago Closed 6 years ago

Upgrade kinto libraries (support of Retry-After headers)


(Toolkit :: Blocklist Policy Requests, defect)

Not set



Tracking Status
firefox53 --- fixed


(Reporter: leplatrem, Assigned: leplatrem)



(2 files, 1 obsolete file)

Currently, if the server fails to process the request, the client will throw and give up. 

With kinto.js v7.0.0 and kinto-http.js v2.7.0, if server provides the `Retry-After` header, the client will wait and retry (once by default).
Attachment #8827439 - Attachment is obsolete: true
Comment on attachment 8827440 [details]
Bug 1331604 - Minor style and idiom changes ()
Attachment #8827440 - Flags: review?(mgoodwin) → review+
Attachment #8827438 - Flags: review?(eglassercamp)
Assignee: nobody → mathieu
Comment on attachment 8827438 [details]
Bug 1331604 - Upgrade kinto libraries ()

Looks fine to me. This is a huge jump, from kinto-http.js 2.0.0 to 2.6.0, and a lot of changes are due to the added support for the attachment plugin. Note that I don't have L3 access, so my review isn't sufficient to autoland this.
Attachment #8827438 - Flags: review?(eglassercamp) → review+
Keywords: checkin-needed
I had to r+ this in MozReview to work around Autoland not seeing it as being ready to land. That said, Autoland couldn't rebase this for pushing, so you'll need to update the patch and re-request checkin.
Keywords: checkin-needed
Ryan, I'm sorry for the trouble :/ 
I rebased and fixed, it should be all good now.
Keywords: checkin-needed
Pushed by
Upgrade kinto libraries (r=glasserc,RyanVM)
Minor style and idiom changes (r=mgoodwin)
Keywords: checkin-needed
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.