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

Missing mime type for otf files (opentype fonts) ? #352

Open
arnaudruffin opened this issue Feb 28, 2023 · 5 comments · Fixed by #358
Open

Missing mime type for otf files (opentype fonts) ? #352

arnaudruffin opened this issue Feb 28, 2023 · 5 comments · Fixed by #358

Comments

@arnaudruffin
Copy link
Contributor

What version of Cloud Foundry and CF CLI are you using? (i.e. What is the output of running cf curl /v2/info && cf version?
CF :

  • api_version 2.195.0
  • osbapi_version 2.15
    CF-CLI: 8.2.0+fd8fbca64.2022-02-09

What version of the buildpack you are using?
staticfile_buildpack_v1_5_36

If you were attempting to accomplish a task, what was it you were attempting to do?
I expose some otf assets

What did you expect to happen?
I would expect them to be exposed with "Content-Type: font/otf" (and with Content-Encoding: gzip)

What was the actual behavior?
The files are served, but with Content-Type: application/octet-stream

I am under the impression that the configuration is missing from mime.types.
I saw the following issue + PR #87 that added the configuration for the other types of font, but not for otf files.

I wanted to check and push a pull request but I can't make out how the mime.types file is generated/retrieved.

@arjun024 arjun024 self-assigned this Mar 10, 2023
@arjun024
Copy link
Member

I don't see otf in the default https://github.com/nginx/nginx/blob/master/conf/mime.types
Are you able to use the nginx-buildpack for this purpose? I'm not very familiar with otf to say it should be included in the staticfile buildpack

@arjun024
Copy link
Member

@ryanmoran @johnnyr0x thoughts?

@arjun024 arjun024 removed their assignment Mar 10, 2023
@arjun024
Copy link
Member

I'm closing this issue under the premise that this is a usecase for nginx buildpack and not staticfile buildpack.
Please feel free to reopen the issue for any further conversation

@ryanmoran
Copy link
Contributor

@arnaudruffin we would happily accept a PR. You can find the mimetypes list here:

@ryanmoran ryanmoran reopened this Mar 17, 2023
@arjun024 arjun024 linked a pull request Apr 5, 2023 that will close this issue
1 task
@ForestEckhardt
Copy link
Member

@arnaudruffin I believe that this was resolved with #358. Is that correct?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

5 participants