cloudinary / cloudinary/cloudinary_ios
For upload, consider using [NSMutableURLRequest setHTTPBodyStream] instead of setHTTPBody
- 主要言語
- Swift
- スター
- 169
- フォーク
- 107
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
Hi there,
I wanted to make a quick suggestion.
Cloudinary can use a significant amount of memory, particularly when uploading videos or large photos because image/video data must be fully loaded into RAM before being sent over the wire.
It looks as though NSMutableURLRequest supports setting an body stream instead of data.
``` obj-c
NSMutableURLRequest *request = [[NSMutableURLRequest alloc] initWithURL:url];
NSInputStream *stream = [[NSInputStream alloc] initWithFileAtPath:filePath];
[request setHTTPBodyStream:stream];
```
This might be something worth looking into to reduce Cloudinary's memory footprint.
Thanks,
Martin
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
調査の方向性
まず、現在 setHTTPBody を使用してアップロードデータを設定している iOS のアップロードリクエスト実装を見つけ、ファイルと動画のアップロードがどのように構築されているかを調べます。既存のリクエストフローを NSMutableURLRequest の setHTTPBodyStream と比較し、完了処理とエラー処理がどのように機能するかを確認します。既存のアップロードを壊すことなく、大容量のアップロードでペイロード全体をメモリ上に保持する必要がなくなれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- ios, objective-c
- 領域
- mobile
- issue の種類
- 機能追加
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 停滞
- 明瞭さ
- 説明が足りない
- 初心者へのやさしさ
- 25/100