review by webdev? if clouserw or morgamic signs off, a=pkim
Sending trunk/includes/header-portal-pages.inc.php Sending trunk/includes/header.inc.php Transmitting file data .. Committed revision 6811. Sending production/includes/header-portal-pages.inc.php Sending production/includes/header.inc.php Transmitting file data .. Committed revision 6812.
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
The script has fired our Lightning Map server. We are tuning up the configuration but need some extinguishers. Can you put the script only on your home page?
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Created attachment 280896 [details] [diff] [review] diff 2 firescript: removed from the include files, added to the home page.
Attachment #280896 - Flags: review?
a) bad idea to have mozilla.com rely on an external site b) the mozilla japan server can't handle the load We are reverting this change asap.
Seems like this would be better as a fake 1x1 image so that it doesn't rely on <script>. ...or even, put the script at the bottom of the page so the rest of the page loads first.
No, we shouldn't be relying on another site.
So... Talked to Kohei and what I told him was there are a couple of possibilities: - backfill the IPs for the mozilla24 map from logs (entirely background) - use an asynchronous request called via setTimeout() (not ieal) But there are privacy questions surrounding logging IPs on a diff site that we haven't even touched on. And there is still the load issue which remains untested. In the future, I'd suggest not pushing something live in the middle of the night -- and I'd like to be cc'd on the actual IT bug?
We will not deploy this again. period. this was a bad idea on many levels.
Status: REOPENED → RESOLVED
Last Resolved: 11 years ago → 11 years ago
Resolution: --- → WONTFIX
Attachment #280896 - Flags: review? → review-
Component: www.mozilla.org/firefox → www.mozilla.org
Product: Websites → Websites
Component: www.mozilla.org → General
Product: Websites → www.mozilla.org
You need to log in before you can comment on or make changes to this bug.