122518 - Speed up the showbuilds.cgi redirect on http://tinderbox.mozilla.org

Status: VERIFIED FIXED

(Webtools Graveyard :: Tinderbox, enhancement)

Description

[Brian 'netdragon' Bober]

I will attach the patch shortly

Comment 1

[Brian 'netdragon' Bober]

-> Me

Comment 2

[Brian 'netdragon' Bober]

Attachment: Diff against index.html

Diff -u from mozilla/.. dir against mozilla/webtools/tinderbox/index.html

Comment 3

[Christian Reis]

This looks fine by me. caillon has suggested using .htaccess to do this at the
header level (lynx doesn't support REFRESH IIRC) but I'm not going to be picky.

Why don't we just kill index.html and make showbuilds.cgi index.cgi?

timeless?

Comment 4

[timeless]

personally i don't like the title change or the body change. you're actually 
making the page longer, which serves very little purpose.

a cgi that spat out a temporary redirect might be better.

mcafee, what do you think?

Comment 5

[Brian 'netdragon' Bober]

Attachment: New Diff per timeless comments

Same deal.

Comment 6

[timeless]

that looks fine. if it's ok w/ mcafee we can do it.

Comment 7

[Chris McAfee]

What is the perceived user change here?
How fast does the page switch now?

Comment 8

[Brian 'netdragon' Bober]

Before, it took 1 second to switch - which was a little aggravating if you were
in a hurry. Now it takes 0 seconds to switch. Besides that, there is no change.
Just as before, if their browser doesn't support the meta tag, they have to
click the link provided.

Comment 9

[Chris McAfee]

r=mcafee.  go ahead and check this in, let me know so I can update the server
on mozilla.org.

Comment 10

[timeless]

fixed.

Comment 11

[Brian 'netdragon' Bober]

mcafee: its checked in

Comment 12

[Chris McAfee]

updated the server, this seems the same to me. ?

Comment 13

[Brian 'netdragon' Bober]

Almost the same. Just a little faster.