fix(backend): オブジェクトストレージの設定でPrefixを設定していなかった場合、nullが文字列として展開・ドメインの1つ…

[1673beta]

What

objectStoragePrefixがnullな際に、https://{domain}/null/{filename}またはhttps://{domain}//{filename}になるのを修正

Why

fix #13963
fix #7613

Additional info (optional)

参考:

• prefixを設定した場合: https://testr2.virtualcat.net/files/webpublic-83dec9dc-c354-4425-a6a5-64ac74f9bb82.webp
• prefixがnullな場合: https://testr2.virtualcat.net/webpublic-1c06bd20-1fc0-4190-ae9a-ed4b72613aff.webp

Checklist

• Read the contribution guide
• Test working in a local environment
• (If needed) Add story of storybook
• (If needed) Update CHANGELOG.md
• (If possible) Add tests

[github-actions]

このPRによるapi.jsonの差分
差分はありません。
Get diff files from Workflow Page

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.

Project coverage is 38.64%. Comparing base (231c2c2) to head (8900599).
Report is 8 commits behind head on develop.

Files with missing lines	Patch %	Lines
packages/backend/src/core/DriveService.ts	0.00%	4 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop   #15432      +/-   ##
===========================================
- Coverage    38.64%   38.64%   -0.01%     
===========================================
  Files         1607     1607              
  Lines       204774   204775       +1     
  Branches      4016     4018       +2     
===========================================
  Hits         79141    79141              
- Misses      124991   124992       +1     
  Partials       642      642              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[fruitriin]

めちゃ歓迎だけど、この影響を受けていた既存のサーバーは今までアップロードされてたやつおかしくなりそう〜〜〜

[1673beta]

このPRをマージしても影響するのは新規にアップロードされるやつだけですね

[syuilo]

Prefixを設定していない状態で保存できないようにする選択肢はないでしょうか？

[1673beta]

admin/update-meta周りかフロントエンドを調整すればできるかもしれませんが、既存のサーバーはnullで登録できていたので、破壊的変更にならないかが懸念点です

[1673beta]

APIの変更は#15454 も含めて別PRで修正しようと思います

[1673beta]

また、オブジェクトストレージのprefixがnullであること自体はS3で許容されていることなので特段不都合がないなら今のPRの変更で問題がないんじゃないかと思います

[mi-gh-maintainer]

Thank you 🙏