cloudinary / cloudinary/cloudinary_ios

For upload, consider using [NSMutableURLRequest setHTTPBodyStream] instead of setHTTPBody

Aperta
#37 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
enhancement
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

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.