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

ファイルサイズについてロール単位で制御できるようにする #14130

Open
KisaragiEffective opened this issue Jul 5, 2024 · 7 comments
Labels
✨Feature This adds/improves/enhances a feature packages/backend Server side specific issue/PR

Comments

@KisaragiEffective
Copy link
Sponsor Member

[ファイルサイズについて] もし単体での上限を実装するときは

ロール単位でのサイズ制限が欲しくて

のほうが融通を利かせやすいかもですね。ちょっと実装量増えるけど…

Originally posted by @samunohito in #11801 (comment)

@KisaragiEffective KisaragiEffective changed the title ふぁもし単体での上限を実装するときは ファイルサイズについてロール単位で制御できるようにする Jul 5, 2024
@KisaragiEffective
Copy link
Sponsor Member Author

変なタイトルで途中送信してしまったのでタイトルを変更

@samunohito
Copy link
Member

念のため。

#11801 (comment)
↑もこのissueのスコープで相違ないです?

@KisaragiEffective
Copy link
Sponsor Member Author

分けることもできると思うけど、一緒にしたほうがスマートなら一緒にするべきだと思う

@samunohito
Copy link
Member

分けるとしても、このissueの前提条件になってくるのと思うので、混ぜちゃっていいような気もします

@kozakura913
Copy link

kozakura913 commented Jul 5, 2024

サーバーの一時保存領域にアップロードされてサイズが確定してから一時保存から永続保存に転送するような実装がされてたから私の実装では転送時に判定してた

@samunohito
Copy link
Member

転送時に判定してた

drive/files/create に対するリクエストのときに

  • クライアント側でリクエストヘッダにContent-Lengthをつけ、サーバ側はそれを使用して判断
  • クライアント側でリクエストボディにサイズを示すプロパティをつけ、サーバ側はそれを使用して判断

のいずれかなイメージですか?

@tamaina
Copy link
Member

tamaina commented Jul 5, 2024

#11801 とは別Issue・別PRにしてください

@KisaragiEffective KisaragiEffective added packages/backend Server side specific issue/PR ✨Feature This adds/improves/enhances a feature labels Jul 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨Feature This adds/improves/enhances a feature packages/backend Server side specific issue/PR
Projects
Status: No status
Development

No branches or pull requests

4 participants