Closed
Bug 1483082
Opened 6 years ago
Closed 6 years ago
Use addons.thunderbird.net instead of a redirect in service preferences.
Categories
(Thunderbird :: Preferences, enhancement)
Thunderbird
Preferences
Tracking
(thunderbird_esr6062+ fixed, thunderbird62 wontfix, thunderbird63 fixed)
RESOLVED
FIXED
Thunderbird 63.0
People
(Reporter: sancus, Assigned: sancus)
Details
Attachments
(1 file)
3.44 KB,
patch
|
jorgk-bmo
:
review+
jorgk-bmo
:
approval-comm-esr60+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → sancus
Assignee | ||
Comment 1•6 years ago
|
||
Assignee | ||
Comment 2•6 years ago
|
||
Comment on attachment 9003959 [details] [diff] [review] Use direct links instead of redirects for blocklist and extensions prefs The reasoning for this is that URLs with a lot of parameters are difficult for Cloudflare to cache, so the blocklists.settings URL and the extensions.getAddons URLs that include a bunch of GUIDs tend to hit the web servers directly. We haven't enabled automatic updating to TB60, and these redirects already make up 45% of our web traffic. Covering this traffic is a silly reason to pay for more web servers, so it's best we just go back to direct links. Is approval-comm-esr60 still the right flag for TB60.1?
Attachment #9003959 -
Flags: review?(jorgk)
Attachment #9003959 -
Flags: approval-comm-beta?
Comment 3•6 years ago
|
||
Comment on attachment 9003959 [details] [diff] [review] Use direct links instead of redirects for blocklist and extensions prefs I don't quite understand it, but r+ anyway. So without this change, "... extensions.getAddons URLs that include a bunch of GUIDs tend to hit the web servers directly". And with the change? Won't direct links have the same effect? And we have to pay for the web servers just the same? Or do you mean we need to pay for the Cloudflare caching which doesn't work in this case, so no need to pay? TB 60 b 11 was built yesterday, but I can get this into TB 60.1.
Attachment #9003959 -
Flags: review?(jorgk)
Attachment #9003959 -
Flags: review+
Attachment #9003959 -
Flags: approval-comm-esr60+
Attachment #9003959 -
Flags: approval-comm-beta?
Assignee | ||
Comment 4•6 years ago
|
||
(In reply to Jorg K (GMT+2) from comment #3) > Comment on attachment 9003959 [details] [diff] [review] > > So without this change, "... extensions.getAddons URLs that include a bunch > of GUIDs tend to hit the web servers directly". And with the change? Won't > direct links have the same effect? And we have to pay for the web servers > just the same? Or do you mean we need to pay for the Cloudflare caching > which doesn't work in this case, so no need to pay?. With the redirect, we hit live.thunderbird.net/blocklists.settings/a/zillion/parameters and then it's redirected to mozilla.com in the case of the blocklist, or the addon servers on AWS for the various extensions.getAddons prefs. That redirect is rarely being cached in Cloudflare because of the zillion parameter problem, so live.thunderbird.net has to serve it. For the services.addons.thunderbird.net URLs, we do have to handle the traffic anyway after the redirect, but this doubles the uncached traffic. And services.addons.thunderbird.net is on AWS and designed to handle a lot of direct traffic from the start. The servers that handle live.thunderbird.net (www. and start. too) are pretty minimal and rely on Cloudflare caching(which is free) to serve >95% of their traffic. All this uncached traffic has been pushing them very close to overloaded on peak euro times(monday mornings). And that's without the TB60.1 auto update, which is another big increase, since these redirects aren't in TB52.
Pushed by mozilla@jorgk.com: https://hg.mozilla.org/comm-central/rev/0d7d34130b9e Use direct links instead of redirects for blocklist and extensions prefs. rs=jorgk CLOSED TREE DONTBUILD
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Comment 6•6 years ago
|
||
Thanks for the explanation. Looks like I understand 43% of it ;-)
Target Milestone: --- → Thunderbird 63.0
Comment 7•6 years ago
|
||
TB 60.1 ESR: https://hg.mozilla.org/releases/comm-esr60/rev/fff4f6c879202bbc217d72252511fe2336ec813c
status-thunderbird62:
--- → wontfix
status-thunderbird63:
--- → fixed
status-thunderbird_esr60:
--- → fixed
Comment 8•6 years ago
|
||
TB 60 beta 11: https://hg.mozilla.org/releases/comm-beta/rev/6ee14490f62a233c4e237c1a9d02d4e6f57efb50
Updated•6 years ago
|
tracking-thunderbird_esr60:
--- → 61+
Updated•6 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•