cloudinary / cloudinary/cloudinary_ios
For upload, consider using [NSMutableURLRequest setHTTPBodyStream] instead of setHTTPBody
- Lingua principale
- Swift
- Stelle
- 169
- Fork
- 107
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
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
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Direzione di ricerca
Inizia individuando l’implementazione della richiesta di upload iOS che attualmente assegna i dati di upload con setHTTPBody, quindi esamina come vengono costruiti gli upload di file e video. Confronta il flusso della richiesta esistente con setHTTPBodyStream di NSMutableURLRequest e determina come funzionano il completamento e la gestione degli errori. Il lavoro è completato quando gli upload di grandi dimensioni non richiedono più l’intero payload in memoria senza compromettere gli upload esistenti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- ios, objective-c
- Ambito
- mobile
- Tipo di issue
- Funzionalità
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Da chiarire
- Idoneità per principianti
- 25/100