1M以上の画像がアップロードできないことに関して

kannokanno commented 11 years ago

タイムラインに流すよりはissueの方が適切かな、という気がしたのでこちらに書きます。

課題

Nginxの設定により1Mを超えるファイルはアップロードできない。これによりアイコン変更では、ユーザーに1M以下のファイルを用意してもらう必要がある。

検討事項

JavaScriptで画像リサイズできるのか？
- nodeやブラウザAPIを使っているならともかく、素のJSで果たして…。
- 仮に無理な場合、別言語でリサイズ用のModuleを作る必要がある
Nginxの制限をいくつまで上げるのか？
クライアント側でリサイズ

HTML5のFileAPIとCanvasを使うと、クライアントJSで「アップロード前にリサイズ」することが可能です。

検討事項

リサイズしたデータはBase64(dataURI)なので、それを送るとするとサーバー側でデコードする必要が出てくる
- 現状のAPI(multipart-form)が変わる

で、何が言いたかったといいますと

本来は小さいはずのアイコンのためだけに、ファイルサイズ制限を引き上げるのは良くない気がしている
- 帯域負荷になり得る変更なので
- クローズド環境なら気にしなくていいじゃん、というのはありますが
無駄な通信量を減らすためにも、「クライアントJSでリサイズしてからアップロードする」方が綺麗なのではと思う
- Androidとか、Web以外のデバイスでも同じように実装してもらう必要が出るのが難点

という一意見です。理想はクライアント/サーバー両方で対応することなんでしょうけどね。

kannokanno commented 11 years ago

ちなみに「クライアントJSでアップロード前にリサイズ」は、手元で軽くやってみたところクライアント側のBase64変換までは出来ました。

が、ローカルだと(上記対応関係なく元から)何故かそもそもアイコン変更ができなくて、そこから先の確認は出来なくて中断しました。

kannokanno commented 11 years ago

理想はクライアント/サーバー両方で対応することなんでしょうけどね。

本文を見直して思いましたが、やっぱりどっちも対応するのがベストか。 (現実解ではなく理想論として)

haushinka2dx commented 11 years ago

現実的に考えて

って感じでしょうか。サーバー側の画像変換は自前で実装はしたくないですねえ。ブラウザとかjQuery に依存してなければ使えるから探してみますかね。

現状のAPI(multipart-form)が変わる

ちなみにBASE64になってしまうのであれば /user/change_avator_base64 って感じでサーバー側のAPIを追加してもいいかもしれないです。

が、ローカルだと(上記対応関係なく元から)何故かそもそもアイコン変更ができなくて、そこから先の確認は出来なくて中断しました。

サーバー側の問題かなと。

パッと思いつくのはこのくらいです。各パスはソースを見てください（すみません）

kannokanno commented 11 years ago

やっぱ権限周りですよねえ。モジュール化していないブランチでもダメだったので、自分も権限周りかなと思います。リクエストが届いているところまではログで確認できているし。