Issue Details (XML | Word | Printable)

Key: CONF-11774
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Critical Critical
Assignee: Matt Ryall [Atlassian]
Reporter: Matt Ryall [Atlassian]
Votes: 0
Watchers: 2
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Confluence

IE6 clients download spacer.gif from wrong URL many times

Created: 08/May/08 08:35 PM   Updated: 30/Jun/08 03:32 AM
Component/s: Performance, Web Interface
Affects Version/s: 2.8
Fix Version/s: 2.8.1

Time Tracking:
Not Specified

Issue Links:
Duplicate
 
Reference
 

Participants: David Taylor [Atlassian], Matt Ryall [Atlassian] and Mike Cannon-Brookes [Atlassian]
Since last comment: 28 weeks ago
Resolution Date: 20/May/08 12:21 AM
Labels:


 Description  « Hide
If Confluence is set up without a context path, there will be a performance impact from requests being served for spacer.gif. This is a particularly large problem when the clients using Confluence are Internet Explorer 6.

The primary problem is that the requests use the wrong URL and hit the (slightly expensive) 404 page:

"GET /display/CONFEXT/undefined/images/border/spacer.gif HTTP/1.1" 404 40448 196468

That's 40 KB download in a response that took 196 ms to generate. By contrast, a 200 response for this file serves 43 bytes in 2 ms and also gets caching headers returned so subsequent requests get a 304 response.

"GET /images/border/spacer.gif HTTP/1.1" 200 43 1632

The other related problem is that IE6 clients request this resource once per comment on a page, rather than once for the whole page.



 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
Mike Cannon-Brookes [Atlassian] added a comment - 16/May/08 05:49 AM
Guys - I'm sure you've considered it, but could we simply have a basic 404 page for any image or CSS requests (which aren't going to visually seen by anyone anyway) across all our apps?

Matt Ryall [Atlassian] added a comment - 18/May/08 10:21 PM - edited
Chris B has a separate issue to dumb down the default 404 page, I believe. I think we're removing the space list from the 404 page entirely, making it almost completely static.

Update: Mike, this is done but not committed yet. The relevant issue is linked, CONF-9050.


David Taylor [Atlassian] added a comment - 20/May/08 12:21 AM
Works fine. Might recommend looking at ways to abstract use of AlphaImageLoader to a single JS helper method in AJS, but not required for release.