cloudinary / cloudinary/cloudinary_ios

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

Offen
#37 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
enhancement
Vorherrschende Sprache
Swift
Sterne
169
Forks
107
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

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

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Beginne damit, die iOS-Implementierung der Upload-Anfrage zu finden, die die Upload-Daten derzeit mit setHTTPBody zuweist, und untersuche anschließend, wie Datei- und Video-Uploads erstellt werden. Vergleiche den bestehenden Anfrageablauf mit setHTTPBodyStream von NSMutableURLRequest und ermittle, wie Abschluss und Fehlerbehandlung funktionieren. Erledigt ist die Aufgabe, wenn große Uploads nicht mehr die vollständige Nutzlast im Speicher benötigen, ohne bestehende Uploads zu beeinträchtigen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
ios, objective-c
Bereich
mobile
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.