-
Notifications
You must be signed in to change notification settings - Fork 308
reset.css doesn't load sometimes #1308
Comments
This is tricky to debug because by the time you open the network tab of the debug console and refresh, the bug goes away. |
I'm gonna assume that's because chrome forces a cache clear when dev tools is opened. No idea how to reproduce though. |
I am definitely hitting this "often." I wrote a script to smash at reset.css a little bit: https://gist.github.com/whit537/18046310e4c6db689741 Here's the result of running that for a while: https://gist.github.com/whit537/91dafe2a6239a44a88c5 It looks like we're alternating between |
I looked at a sniffer:
HTTPScoop works great but it can't read SSL. They recommend setting up a reverse proxy: |
Apparently Wireshark can be used to decrypt SSL: |
I use Fiddler normally, it just installs a certificate proxy so you can decrypt and view. |
Windows only. :( |
@clone1018 Next time you see this can you grab a capture with Fiddler? I find that a simple refresh gives me the same missing-reset behavior. |
We could probably fix this by moving reset.css to the CDN but it'd be better to get to the bottom of this if we can. |
Eureka! For 304 the content-type is text/html! Found using http://www.charlesproxy.com/, per @greggles. |
Here's where Tornado dealt with this: https://groups.google.com/forum/#!topic/python-tornado/-P_enYKAwrY Looks like it's safe to simply unset My apologies to Cheroot. :-) |
Don't send Content-Type for 304; fixes #1308
@clone1018 saw this when we first started caching static assets. It's why I turned off static caching initially. Now static caching is back with #1245 and indeed we're seeing this again. :(
The text was updated successfully, but these errors were encountered: