Skip to content

/files/ 下のヘッダ設定タイミングを修正

Kainoa Kanter requested to merge gh-fa55fa5e/5650/unknown/refs/pull/5650/head into develop

Summary

/files/以下のアクセスで500になってもCache-Control: max-age=31536000, immutableが送られてしまったりするのを修正

immutableの一括指定 の修正

最初に/files/以下をctx.set('Cache-Control', 'max-age=31536000, immutable'); → 例外で500になっても送られてしまうので 送出時に設定するように修正

Content-Type指定の後のbody設定 の修正

ctx.set('Content-Type', 'image/jpeg');
ctx.body = file;

→ body設定タイミングでContent-Typeが自動設定されることがあるのでContent-Type設定は後に

body読み系処理の前のヘッダ－設定 の修正

ctx.set('Content-Type', file.type === 'image/apng' ? 'image/png' : file.type);
ctx.set('Content-Disposition', contentDisposition('inline', `${rename(file.name, { suffix: '-web' })}`));
ctx.body = InternalStorage.read(key);

→ 例外で500になっても送られてしまうので 後で設定するように修正

Merge request reports