Skip to content
This repository has been archived by the owner. It is now read-only.

fonts.css 404s for some UAs #931

Closed
ckarlof opened this issue Apr 16, 2014 · 7 comments
Closed

fonts.css 404s for some UAs #931

ckarlof opened this issue Apr 16, 2014 · 7 comments

Comments

@ckarlof
Copy link
Contributor

@ckarlof ckarlof commented Apr 16, 2014

Investigating now, but it may be related to how connect-fonts caches generated fonts.

@ckarlof
Copy link
Contributor Author

@ckarlof ckarlof commented Apr 16, 2014

screen shot 2014-04-16 at 2 17 32 pm

screen shot 2014-04-16 at 2 18 52 pm

@ckarlof
Copy link
Contributor Author

@ckarlof ckarlof commented Apr 16, 2014

Here's the code I suspect: https://github.com/shane-tomlinson/connect-fonts/blob/d4134e35e2e9d8fe82f3a9378ef25f84807dfb78/lib/css-responder.js#L166

connect-fonts generates and caches fonts based on ua, requested fonts, and locale. One way we could get 404s is that that connect-fonts thinks it has a cached font, but there is no matching file on disk.

I suspect a server restart would resolve this, but let's not do that yet.

@vladikoff
Copy link
Contributor

@vladikoff vladikoff commented Apr 16, 2014

Change the user agent to something "unique" helps generate the font-file

Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:28.0) Gecko/20100101 Firefox/999.0

i.e Firefox 999
image

@ckarlof
Copy link
Contributor Author

@ckarlof ckarlof commented Apr 16, 2014

Yeah, we're removing this magic in train-10, so maybe we just need to get train-10 out. :) I have some questions about our release process for @mostlygeek, though, before we do that.

@ckarlof
Copy link
Contributor Author

@ckarlof ckarlof commented Apr 16, 2014

Leaving this open until we can verify it's fixed on train-10 release, but nothing to worry about now. We'll get Helvetica until train-10 goes live and love it.

@pdehaan pdehaan added this to the train-11 (Apr 21) milestone Apr 21, 2014
@pdehaan
Copy link
Contributor

@pdehaan pdehaan commented Apr 21, 2014

This can be closed after train-10 leaves the station. #choochoo

@zaach
Copy link
Contributor

@zaach zaach commented Apr 22, 2014

Train 10 is live. 🚋 🚋 🚋

@zaach zaach closed this Apr 22, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants