Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

画像の圧縮(webpublic化)をクライアントサイドで行う #8173

Closed
tamaina opened this issue Jan 22, 2022 · 2 comments · Fixed by #8216
Closed

画像の圧縮(webpublic化)をクライアントサイドで行う #8173

tamaina opened this issue Jan 22, 2022 · 2 comments · Fixed by #8216
Labels
✨Feature This adds/improves/enhances a feature

Comments

@tamaina
Copy link
Contributor

tamaina commented Jan 22, 2022

Summary

#7704 (comment)
#8167 (comment)

#8167 (webpublicのみ保持する) 場合、画像の圧縮はブラウザサイドで行い、サーバーは圧縮作業をパスする。

画像圧縮は現代の半導体を使っていれば負荷は気にならないと思う。
一方でアップロード時の通信量を圧縮できるのは大きい。

圧縮作業のパススルーについて

アップロード時にisWebpublicとしてwebpublicとして圧縮済みであるフラグを付けるのもいいけど、サードパーティアプリが適切なファイルを送ってくるとは限らないと思う。

そこで、generateAltsで「元データがwebpublicとして使えるか検査し、使えるようであればwebpublicをnullにする」のような処理を作ればwebpublicであることを確実にしつつ圧縮を省略できると思った。

@tamaina tamaina added the ✨Feature This adds/improves/enhances a feature label Jan 22, 2022
@mei23
Copy link
Contributor

mei23 commented Jan 22, 2022

「元データがwebpublicとして使えるか検査し、使えるようであればwebpublicをnullにする」

えー、1回開いてサイズ/orientation/EXIFまで見るのかしら
→ でも、どうせサムネのために1回は開くから別にいいのかも

@tamaina
Copy link
Contributor Author

tamaina commented Jan 22, 2022

うん、generateAltsを実行すれば一回は開くので処理が増えるわけではなさそう

https://github.com/syuilo/misskey/blob/59520c31fde073f7cb0ede81162898f2bc340b4f/packages/backend/src/services/drive/add-file.ts#L184

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment