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

Enabling WOFF files under Windows IIS #11091

Merged
merged 2 commits into from Dec 26, 2016
Merged

Conversation

ksubileau
Copy link
Contributor

Hi,
After upgrading to Piwik 3 on Windows IIS, I encountered a 404 error on the icon file (/plugins/Morpheus/fonts/piwik.woff) because the type for WOFF files (application/font-woff) isn't included by default in the IIS server configuration (similar issues has been reported previously on IIS (#5255) but also on Apache (#8691, #9967)).

To fix this, I suggest to add the appropriate mime type in the IIS configuration file.

(Off-topic) By the way, in the ServerFilesGenerator class, why the createWebConfigFiles method is protected whereas the deleteWebConfigFiles and createHtAccessFiles (and others) methods are public ? This seems inconsistent and forces me to use the createFilesForSecurity method, which will also recreate the Apache files.

@mattab
Copy link
Member

mattab commented Dec 26, 2016

Thank you very much! I had seen these reports and didn't realise we could fix this problem in Piwik. Awesome!

By the way, in the ServerFilesGenerator class, why the createWebConfigFiles method is protected whereas the deleteWebConfigFiles and createHtAccessFiles (and others) methods are public ? This seems inconsistent and forces me to use the createFilesForSecurity method, which will also recreate the Apache files.

Ideally all methods except createFilesForSecurity should be protected and we'd always have to call createFilesForSecurity (for simplicity)

@mattab mattab merged commit 490363a into matomo-org:3.x-dev Dec 26, 2016
@ksubileau ksubileau deleted the woff_iis branch December 27, 2016 22:16
@Findus23 Findus23 mentioned this pull request Apr 4, 2018
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.

None yet

2 participants