improve YSlow! and PageSpeed score of dekiwiki

RESOLVED FIXED in 0.9.3

Status

developer.mozilla.org
General
RESOLVED FIXED
7 years ago
6 years ago

People

(Reporter: groovecoder, Assigned: craigcook)

Tracking

unspecified
0.9.3
x86
Mac OS X

Details

(Reporter)

Description

7 years ago
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/*
(Reporter)

Updated

7 years ago
Blocks: 607658
(Assignee)

Comment 1

7 years ago
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.
(Reporter)

Comment 2

7 years ago
(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/
(Assignee)

Comment 3

7 years ago
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.
(Reporter)

Comment 4

7 years ago
(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?
(Reporter)

Comment 5

7 years ago
(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. :(
(Reporter)

Updated

7 years ago
Assignee: nobody → craigcook.bugz
(Assignee)

Comment 6

7 years ago
Moved .htaccess to mdn and mozilla in r82501, added to common in r82603
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
(Reporter)

Comment 7

7 years ago
(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?
(Assignee)

Comment 8

7 years ago
(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).
Component: Deki Infrastructure → Other
Product: Mozilla Developer Network → Mozilla Developer Network
You need to log in before you can comment on or make changes to this bug.