Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix for #85 #86

Closed
wants to merge 1 commit into from
Closed

Fix for #85 #86

wants to merge 1 commit into from

Conversation

vendruscolo
Copy link
Contributor

Cache rules have been replicated in the latter webfont location block.
We now have just one location block for webfonts, that contains CORS
rules and cache rules.

There were two location blocks for webfonts, so
`Access-Control-Allow-Origin` never matched.
Cache rules have been replicated in the latter webfont location block.
We now have just one location block for webfonts, that contains CORS
rules **and** cache rules.
@AD7six AD7six closed this in 15a669a Aug 19, 2012
alrra pushed a commit to h5bp/server-configs-nginx that referenced this pull request Jul 28, 2013
Only one location match is ever executed - therefore the relevant
location match must contain all relevant rules.

cross-domain-fonts is included by default, therefore for right now add
the cache rules for these requests directly to the
cross-domain-fonts.conf file

fixes h5bp/server-configs#85
fixes h5bp/server-configs#86

Thanks to @misterjack
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants