fjordllc / bootcamp

プログラマー向けEラーニングシステム
https://bootcamp.fjord.jp
MIT License
286 stars 71 forks source link

画像のバリデーションをMimeTypeを用いるものに変更 #8092

Open reckyy opened 1 month ago

reckyy commented 1 month ago

Issue

概要

riraさんが作成したPRの概要から引用させていただきます。

アバター画像のバリデーションにMarcelを使ったMime-Typeのチェックを加えました。 受け入れるべきファイル形式であるかをファイルの実体から判別するようになるため、拡張子偽装したファイルや形式判別不可能なファイルをアバターとして登録することを防ぐことができます。

変更確認方法

  1. fix/broken-images-can-be-registered をローカルに取り込む
  2. 任意のユーザーでログインする
  3. http://localhost:3000/current_user/edit にアクセス
  4. 壊れた画像をアバター画像に設定し、更新ができない(エラーが出る)ことを確認。

Screenshot

変更前

スクリーンショット_2024-09-27_11_18_54

変更後

スクリーンショット_2024-09-27_11_19_14

reckyy commented 1 month ago

@MikotoMakizuru お疲れ様です! レビューをお願いしたくご連絡いたしました。急ぎではありません。 もしご都合厳しければ、教えていただけますと幸いです。 よろしくお願いいたします。

MikotoMakizuru commented 1 month ago

@reckyy 今から見ます〜

MikotoMakizuru commented 1 month ago

@reckyy 1点コメントさせていただきました〜