558499 - SUMO: Remove Omniture tags

Status: VERIFIED FIXED

(support.mozilla.org :: Knowledge Base Software, task, P1)

Description

[[:Cww]]

+++ This bug was initially created as a clone of Bug #558105 +++
Remove Omniture tags after we're done migrating to Webtrends.  See bug 558105 for justification for splitting the tasks.

Comment 1

[[:Cww]]

So Omniture relies on an SSL cert that we're not renewing and which expire 5/21.  That means that users on secure pages (like the login page) will see SSL cert warnings so we need to have the omniture tags removed by then.

Comment 2

[James Socol [:jsocol, :james]]

Since this is a ~4 line fix and we have outside pressure (and 1.5.5 is tiny right now) this is fine for 1.5.5 or 1.5.4.1 or whatever we need to call it.

With only this bug (we'll clone the 1.5.4 tag) can we push on the 20th or should we aim for the 18th? (Remember we have new people starting next week.)

Comment 3

[Stephen Donner [:stephend] Not actively reading bugmail]

FWIW, from QA's perspective, we can push whenever.

Comment 4

[Paul Craciunoiu [:paulc]]

Attachment: v1, remove omniture js

Comment 5

[Paul Craciunoiu [:paulc]]

Just adding a note here - let's make sure that showfor still works after we remove omniture (see bug 565945).

Comment 6

[James Socol [:jsocol, :james]]

Comment on attachment 445170 [details] [diff] [review]
v1, remove omniture js

There's still a SiteCatalyst script tag hanging out in some template that's causing JS errors on the home page (at least).

Comment 7

[Stephen Donner [:stephend] Not actively reading bugmail]

We need this landed ASAP if QA is going to finish testing and feel confident about the 4pm push.

Comment 8

[James Socol [:jsocol, :james]]

Comment on attachment 445170 [details] [diff] [review]
v1, remove omniture js

Paul pointed me to flushing memcache, and offending <script> tag is gone.

Comment 9

[Paul Craciunoiu [:paulc]]

r67456 on trunk. Once QA verifies, I'll create a 1.5.4.1 tag and get it on there too.

Comment 10

[Rebecca Billings [:rbillings]]

Verified on support-stage-new with firebug and live http headers.