fix(frontend): 拡張子内にドットが含まれるものを正常にアップロードできるように

[kakkokari-gtyih]

What

ランダム文字列でアップロードする際に、tar.gzなど、拡張子内にドットが含まれるファイルの拡張子が正常に処理されない問題を修正

Why

Fix #13688 (comment)

Additional info (optional)

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

[codecov]

Codecov Report

Attention: Patch coverage is 51.85185% with 13 lines in your changes are missing coverage. Please review.

Project coverage is 77.91%. Comparing base (f6df940) to head (f261a73).
Report is 2 commits behind head on develop.

Files	Patch %	Lines
packages/frontend/src/scripts/upload.ts	51.85%	13 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##           develop   #13699       +/-   ##
============================================
+ Coverage    66.46%   77.91%   +11.45%     
============================================
  Files          988      185      -803     
  Lines       116719    25566    -91153     
  Branches      4511      487     -4024     
============================================
- Hits         77572    19920    -57652     
+ Misses       39116     5639    -33477     
+ Partials        31        7       -24     

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

[syuilo]

対応必要？

[kakkokari-gtyih]

filename.xxx.yyy みたいなファイルをアップロードするときには必要（現状.yyyだけが抽出される）

[syuilo]

filename.xxx.yyy が aaaaaaa.yyy になって困ること少ない気もする

[syuilo]

困る場合は設定をオフにするとか

[syuilo]

例外の拡張子を列挙するのは管理が面倒になるからあまりやりたくないわね

[syuilo]

filename.xxx.yyy が aaaaaaa.yyy になって困ること少ない気もする

例えば aaaa.jpg.webp とかは結局webpだから bbbb.webp になっても問題ないし

[kakkokari-gtyih]

（個人的にはそもそもドライブにtar.gzをアップロードするユースケースが思いつかないので、それ以外に困るパターンがなさそうなら別にどっちでも良いとはおもっている）

[syuilo]

ピリオドでsplitした最初だけ置き換えるのはダメなのかしら

[kakkokari-gtyih]

ピリオドでsplitした最初だけ置き換えるのはダメなのかしら

いまそうなってるはず

[syuilo]

ピリオドでsplitした最初だけ置き換えるのはダメなのかしら

いまそうなってるはず

であれば

ランダム文字列でアップロードする際に、tar.gzなど、拡張子内にドットが含まれるファイルの拡張子が正常に処理されない

は起きえない気がした

[kakkokari-gtyih]

今の実装はこれ

[kakkokari-gtyih]

ピリオドでsplitした最初だけ置き換えるのはダメなのかしら

いまそうなってるはず

あ、そうじゃなかった
最後だけ抽出していた

[kakkokari-gtyih]

ピリオドでsplitした最初だけ置き換えるのはダメなのかしら

ランダム文字列でアップロードする際に、tar.gzなど、拡張子内にドットが含まれるファイルの拡張子が正常に処理されない

は起きえない気がした

ただそうなった場合 file.name.that.has.many.dots.mp4 とかが 8fd7940a-e859-47a6-988c-5c67cfcba226.name.that.has.many.dots.mp4 とかになって変に処理されそう

[syuilo]

ピリオドでsplitした最初だけ置き換えるのはダメなのかしら

ランダム文字列でアップロードする際に、tar.gzなど、拡張子内にドットが含まれるファイルの拡張子が正常に処理されない

は起きえない気がした

ただそうなった場合 file.name.that.has.many.dots.mp4 とかが 8fd7940a-e859-47a6-988c-5c67cfcba226.name.that.has.many.dots.mp4 とかになって変に処理されそう

一般的にファイルシステムにおけるファイル名に拡張子を意味する目的以外でピリオドって使われるものなのかしら

[kakkokari-gtyih]

許容されないわけではないのでまずそう

[syuilo]

非常に稀な気がするけどどうなのかしら(特に英語圏)

[kakkokari-gtyih]

ユーザーがどんなファイル名のファイルをアップロードするかはこちらでコントロールできることではないのでユーザーの行動を信用しないほうが良い気はする

[Sayamame-beans]

ファイル名に日付が入っているケース(例 VRChat_2024-04-13_00-40-23.477_2560x1440.png)などは見かけますね…
(初めのドットで切るとrandomname.477_2560x1440.pngになる)

[syuilo]

🤯