misskey icon indicating copy to clipboard operation
misskey copied to clipboard

enhance(server): メディアプロキシでico,bmpを読めるように

Open tamaina opened this issue 2 years ago • 1 comments

Fix #10120

Requres #10150

https://github.com/misskey-dev/sharp-read-bmp を作成し、メディアプロキシでico,bmpを読めるようにした

tamaina avatar Mar 03 '23 17:03 tamaina

Codecov Report

Merging #10186 (25dba9b) into develop (2d551a8) will increase coverage by 0.00%. The diff coverage is 30.00%.

@@           Coverage Diff            @@
##           develop   #10186   +/-   ##
========================================
  Coverage    69.23%   69.24%           
========================================
  Files          707      707           
  Lines        65404    65407    +3     
  Branches      5213     5213           
========================================
+ Hits         45283    45290    +7     
+ Misses       20121    20117    -4     
Impacted Files Coverage Δ
packages/backend/src/server/FileServerService.ts 19.05% <12.50%> (+0.17%) :arrow_up:
packages/backend/src/misc/is-mime-image.ts 100.00% <100.00%> (ø)
packages/backend/src/postgres.ts 95.04% <0.00%> (+0.90%) :arrow_up:
packages/backend/src/logger.ts 79.24% <0.00%> (+1.88%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

codecov[bot] avatar Mar 03 '23 17:03 codecov[bot]

テスト追加できそう?

syuilo avatar Mar 04 '23 08:03 syuilo

startServerしてテストするのは書けると思う

(サービスを直接引っ張ってきて云々するのは書けないっす)

tamaina avatar Mar 04 '23 08:03 tamaina

ただ本体にテスト追加する前にmedia-proxy側でテスト書く(メディアプロキシ周りはいつもその順序でコードを書いている)

tamaina avatar Mar 04 '23 08:03 tamaina

🙏

syuilo avatar Mar 04 '23 10:03 syuilo

(media-proxyにローカルファイル検索関数を差し込めればコードがダブってるの解消するんじゃねって思い始めた)

tamaina avatar Mar 04 '23 10:03 tamaina

downloadUrlなどなどの統合も必要

tamaina avatar Mar 04 '23 10:03 tamaina