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

mp4のふりをしたmp4じゃない動画を弾いている #668

Closed
ikura-hamu opened this issue Aug 27, 2023 · 3 comments · Fixed by #1083
Closed

mp4のふりをしたmp4じゃない動画を弾いている #668

ikura-hamu opened this issue Aug 27, 2023 · 3 comments · Fixed by #1083
Assignees

Comments

@ikura-hamu
Copy link
Member

動画の編集ソフトなどによっては、mp4という拡張子でも実際はmp4ではなく、でもプレイヤーで再生できる動画ができたりする。これをアップロードしようとすると弾かれる。

フロントと相談しながら仕様を決める必要がある。

@mazrean
Copy link
Member

mazrean commented Aug 27, 2023

現在はこれに合致するファイルのみをmp4と判定して通している。
https://github.com/h2non/filetype/blob/v1.1.3/matchers/video.go#L93-L124
コード書くときにMPEG-4のバイナリフォーマット確認して仕様にちゃんと沿っているなら通るか確認したはずだけど、もしかしたら使用に沿っていても通らない場合があるのかも?

@mazrean
Copy link
Member

mazrean commented Aug 27, 2023

あとはm4vとかmkvとかも問題なく再生できるだろうし、通しちゃってもよい気はする(ただし、DBに保存するファイル情報は分けておいた方が良いと思う)

@ikura-hamu ikura-hamu self-assigned this Dec 16, 2024
@ikura-hamu
Copy link
Member Author

m4vとmkvを通すようにする

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 a pull request may close this issue.

2 participants