cloudinary / cloudinary/cloudinary_ios
For upload, consider using [NSMutableURLRequest setHTTPBodyStream] instead of setHTTPBody
- 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