From d5f5f4023c479bbca1ad9e88caa0920359b43ef6 Mon Sep 17 00:00:00 2001 From: tamaina Date: Tue, 28 Feb 2023 16:06:23 +0000 Subject: [PATCH] 0.0.15 --- SPECIFICATION.md | 5 ++++- built/index.js | 6 ++---- package.json | 2 +- src/index.ts | 11 +++++++---- 4 files changed, 14 insertions(+), 10 deletions(-) diff --git a/SPECIFICATION.md b/SPECIFICATION.md index 2345ed1..39ecc04 100644 --- a/SPECIFICATION.md +++ b/SPECIFICATION.md @@ -26,7 +26,7 @@ Acceptヘッダーは無視される。 Cache-Controlは、正常なレスポンスの場合`max-age=31536000, immutable`、エラーレスポンスの場合`max-age=300`である。 Content-Typeは、ファイルの内容について適切なものが挿入される。 Content-Security-Policyは、`default-src 'none'; img-src 'self'; media-src 'self'; style-src 'unsafe-inline'`となっている。 -Content-Dispositionは、filenameは元画像のContent-Disposition.filenameもしくはファイル名に基づいて挿入される。inlineが指定される。 +Content-Dispositionは、filenameは元画像のContent-Disposition.filenameもしくはファイル名に基づいて挿入される。拡張子は適宜変更され、octet-streamの場合は拡張子として.unknownが付加される。inlineが指定される。 ### クエリの一覧 #### url (必須) @@ -52,6 +52,9 @@ https://www.google.com/images/errors/robot.png をプロキシする場合: 変換形式が指定されていなかった場合は、画像ファイルもしくは許可されたファイル（FILE_TYPE_BROWSERSAFE）である場合のみプロキシ（ファイルの再配信）が行われる。 ただし、svgは、webpに変換される（最大サイズ2048x2048）。 +#### 変換クエリ付加時の挙動 +一方、以下の変換クエリが指定されているが、元ファイルがsharp.jsで変換できない形式の場合、404が返される。 + #### emoji 存在すると、高さ128px以下のwebpが応答される。 ただし、sharp.jsの都合により、元画像がapngの場合は無変換で応答される。 diff --git a/built/index.js b/built/index.js index 531263f..07a0fa7 100644 --- a/built/index.js +++ b/built/index.js @@ -183,7 +183,7 @@ async function proxyHandler(request, reply) { } reply.header('Content-Type', image.type); reply.header('Cache-Control', 'max-age=31536000, immutable'); - reply.header('Content-Disposition', contentDisposition('inline', file.filename)); + reply.header('Content-Disposition', contentDisposition('inline', correctFilename(file.filename, image.ext))); return reply.send(image.data); } catch (e) { @@ -210,9 +210,7 @@ async function downloadAndDetectTypeFromUrl(url) { } } function correctFilename(filename, ext) { - if (!ext) - return filename; - const dotExt = `.${ext}`; + const dotExt = ext ? `.${ext}` : '.unknown'; if (filename.endsWith(dotExt)) { return filename; } diff --git a/package.json b/package.json index e734c08..80d8831 100644 --- a/package.json +++ b/package.json @@ -1,6 +1,6 @@ { "name": "misskey-media-proxy", - "version": "0.0.14", + "version": "0.0.15", "description": "The Media Proxy for Misskey", "main": "built/index.js", "packageManager": "pnpm@7.26.0", diff --git a/src/index.ts b/src/index.ts index 0956515..314bfa1 100644 --- a/src/index.ts +++ b/src/index.ts @@ -231,7 +231,12 @@ async function proxyHandler(request: FastifyRequest<{ Params: { url: string; }; reply.header('Content-Type', image.type); reply.header('Cache-Control', 'max-age=31536000, immutable'); - reply.header('Content-Disposition', contentDisposition('inline', file.filename)); + reply.header('Content-Disposition', + contentDisposition( + 'inline', + correctFilename(file.filename, image.ext) + ) + ); return reply.send(image.data); } catch (e) { if ('cleanup' in file) file.cleanup(); @@ -261,9 +266,7 @@ async function downloadAndDetectTypeFromUrl(url: string): Promise< } function correctFilename(filename: string, ext: string | null) { - if (!ext) return filename; - - const dotExt = `.${ext}`; + const dotExt = ext ? `.${ext}` : '.unknown'; if (filename.endsWith(dotExt)) { return filename; }