632995 - improve YSlow! and PageSpeed score of dekiwiki

Status: RESOLVED FIXED

(developer.mozilla.org Graveyard :: General, defect)

Description

[Luke Crouch [:groovecoder]]

Currently 79 for YSlow! and 78 for PageSpeed

* Add Etag and Expires .htaccess to skins/ and/or skins/mdn (see https://github.com/fwenzel/mdn/blob/master/media/.htaccess)
* Combine & Minify skins/mdn/Transitional/js/*

Comment 1

[Craig Cook (:craigcook)]

I can combine and compress the JavaScript easily enough, but the Etag and Expires stuff is outside my area. I'll have to defer to someone who knows more about servers than I do. It seems to be more than just posting the .htaccess file, and I don't know what modules are installed or active or what other server config bits have to happen to make it work properly.

I'll handle the JavaScript today and we may need to loop in someone from server ops for the rest.

Comment 2

[Luke Crouch [:groovecoder]]

(In reply to comment #1)
> I can combine and compress the JavaScript easily enough, but the Etag and
> Expires stuff is outside my area. I'll have to defer to someone who knows more
> about servers than I do. It seems to be more than just posting the .htaccess
> file, and I don't know what modules are installed or active or what other
> server config bits have to happen to make it work properly.
> 
> I'll handle the JavaScript today and we may need to loop in someone from server
> ops for the rest.

the dekiwiki directory should honor .htaccess files so I think we can just copy the same .htaccess file from the django media/ directory (linked in desc) into the dekiwiki skins/ directory and it should work. I can take a stab at it but I don't think I have write access to https://svn.mozilla.org/projects/deki/trunk/mozilla/skins/

Comment 3

[Craig Cook (:craigcook)]

Well just adding the file is easy enough, but when I try it locally I get 500 errors. It seems mod_expires isn't installed in the VM provided by Mindtouch (because Deki is a diva app and that's about the only way to run it locally) but I have no idea what the server setup is on stage9 or production. 

So I've added the file in r82363 and we'll just need someone to install mod_expires if it's not there already.

Comment 4

[Luke Crouch [:groovecoder]]

(In reply to comment #3)
> Well just adding the file is easy enough, but when I try it locally I get 500
> errors. It seems mod_expires isn't installed in the VM provided by Mindtouch
> (because Deki is a diva app and that's about the only way to run it locally)
> but I have no idea what the server setup is on stage9 or production. 
> 
> So I've added the file in r82363 and we'll just need someone to install
> mod_expires if it's not there already.

I think it's there. We have the same .htaccess under the same vhost on stage and it works fine. How do we push dekiwki skin changes to staging? another IT bug?

Comment 5

[Luke Crouch [:groovecoder]]

(In reply to comment #3)
> Well just adding the file is easy enough, but when I try it locally I get 500
> errors. It seems mod_expires isn't installed in the VM provided by Mindtouch
> (because Deki is a diva app and that's about the only way to run it locally)
> but I have no idea what the server setup is on stage9 or production. 
> 
> So I've added the file in r82363 and we'll just need someone to install
> mod_expires if it's not there already.

Looks like we need to move the file from skins/ to skins/mdn and copy it to skins/mozilla. Staging (& prod?) sym-link at the mdn/ and mozilla/ level. :(

Comment 6

[Craig Cook (:craigcook)]

Moved .htaccess to mdn and mozilla in r82501, added to common in r82603

Comment 7

[Luke Crouch [:groovecoder]]

(In reply to comment #6)
> Moved .htaccess to mdn and mozilla in r82501, added to common in r82603

How do we update dekiwiki9 with f82603? Is that *another* IT bug or will dekiwiki auto-update eventually?

Comment 8

[Craig Cook (:craigcook)]

(In reply to comment #7)
> How do we update dekiwiki9 with f82603? Is that *another* IT bug or will
> dekiwiki auto-update eventually?

It should auto-update from SVN but if it hasn't I wonder if the /common/ folder isn't set up to sync (I assumed all of /skins/ was but maybe not).