saekitominaga / media.w0s.jp Goto Github PK

サムネイル画像存在判定を fs.existsSync() で行っているせい

• type[]=xxx;type[]=webp → 正しくエラーになる
• type[]=webp;type[]=xxx → エラーにならない

ファイルパス、タイプ、画像幅&高さ、画質が完全一致したデータのみユニーク制約ではじくべきところ、ファイルパスに主キーが設定されているためが同じものはすべてはじかれてしまう。

• 別サイトから <img> 等で埋め込まれるのを防止
• IE 以外の全ブラウザが対応 Can I use...

T/O

• Class の constructor 内は除外（await が使えないので）
• existsSync は除外

#10 で作成した Validator クラスで指定されているパラメーター名が間違っていて、常にエラーになる。（2021年10月以降ファイルアップロードをしていなかったのでこれまで気付かなかった）

#2 の対応の一環

thumbimage に合わせる

#2 の対応の一環

type の複数指定に関する記述がされていないなど若干古い状態なので、この際全体的に見直す。

ThumbImageCreateValidator と ThumbImageRenderValidator はクラス（ファイル）を統合し、メソッドで区別する。

SaekiTominaga/w0s.jp#39 に合わせた改修（本サイトでは当該バグは発生していないが、極力コードを合わせるために改修する）

/blogupload → /blog/upload

#2 の前段階の作業として

バッチ処理が行われる前に手動でサーバーから画像を削除した場合（これは普通にあり得る）、現状は 403 を返しているのでバッチ処理がエラー扱いで終了する。
次も同じリクエストをしてくるので、永遠に同じエラーを返す状態になってします。
レスポンスコードは 200 とし、必要ならボディで「画像が存在しない」情報を返すようにすべき。

#8 で AVIF 生成時のみ DB アクセスを行うようにしたため、以下の全ての条件が満たされた場合に Error: SQLITE_BUSY: database is locked の例外が発生する。

• 多数の <img> が埋め込まれたページ
• サムネイル画像がまだ生成されていない
• AVIF 対応ブラウザ（Firefox, Chrome）でアクセス

SaekiTominaga/w0s.jp#5

T/O

• 1ページ内の <img> の数が多いと、生成に分単位の時間が掛かり、 CPU も圧迫する
• JPEG （WebP も?）ならそこそこ早いので、仮に JPEG 等で返しておき、 AVIF の生成リストを DB に保存するなどしてバッチ処理で別途生成の形にできないか