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(common/helper): the mimetype guesser is not good at recognize text file #1083

Merged
merged 3 commits into from
Nov 28, 2024

Conversation

theus77
Copy link
Member

@theus77 theus77 commented Nov 27, 2024

Q A
Bug fix? y
New feature? n
BC breaks? n
Deprecations? n
Fixed tickets? n
Documentation? n

it's better to use the file extension to recognise the mime-type of a text file

@theus77 theus77 marked this pull request as ready for review November 27, 2024 16:06
@theus77 theus77 requested a review from a team as a code owner November 27, 2024 16:06
Copy link
Member

@Davidmattei Davidmattei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Phpstan fails

@theus77 theus77 changed the title fix: the mimetype guesser is not good as recognise text file fix: the mimetype guesser is not good at recognize text file Nov 27, 2024
@Davidmattei Davidmattei added this to the 5.24.1 milestone Nov 28, 2024
@Davidmattei Davidmattei changed the title fix: the mimetype guesser is not good at recognize text file fix(common/helper): the mimetype guesser is not good at recognize text file Nov 28, 2024
@Davidmattei Davidmattei merged commit 98dc5c0 into ems-project:5.24 Nov 28, 2024
3 checks passed
@Davidmattei Davidmattei added bug Something isn't working common-bundle labels Nov 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working common-bundle
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants