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

nginx: fix missing ttf static files #3229

Merged
merged 2 commits into from
Jan 18, 2024

Conversation

verymilan
Copy link
Contributor

in order to fix broken icons after the .2 release :)

@prolibre
Copy link

Hello @verymilan !

So I'm working on it because I'm using Apache and I need to adapt the code.

I've put this :
"(?i)\.(gif|bmp|ico|jpg|jpeg|png|tif|tiff|webp|css|js|ttf|svg|woff|scss)$"

based on what I found in images and static files. The (?i) is because I also found capital letters.

@hughrun
Copy link
Contributor

hughrun commented Jan 18, 2024

@verymilan sorry this was my fault. Do you mind adding svg as well? See #3230

@verymilan verymilan changed the title nginx: fix missing ttf static files nginx: fix missing ttf static files, fixes #3230 Jan 18, 2024
@verymilan verymilan changed the title nginx: fix missing ttf static files, fixes #3230 nginx: fix missing ttf static files Jan 18, 2024
@mouse-reeve mouse-reeve merged commit fd0b1d9 into bookwyrm-social:main Jan 18, 2024
10 checks passed
@marcusyoung
Copy link

I also had to add woff|scss|gif|webp to prevent 404 errors. These don't seem to have made it into the nginx config file in the production branch, which still only has: location ~ \.(bmp|ico|jpg|jpeg|png|svg|tif|tiff|webp)$

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.

5 participants