cloudinary / cloudinary/cloudinary_ios
For upload, consider using [NSMutableURLRequest setHTTPBodyStream] instead of setHTTPBody
- Langage dominant
- Swift
- Étoiles
- 169
- Forks
- 107
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
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
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Piste de recherche
Commencez par localiser l’implémentation de la requête d’upload iOS qui affecte actuellement les données d’upload avec setHTTPBody, puis examinez comment les uploads de fichiers et de vidéos sont construits. Comparez le flux de requête existant avec setHTTPBodyStream de NSMutableURLRequest et déterminez comment fonctionnent la finalisation et la gestion des erreurs. Le travail est terminé lorsque les gros uploads ne nécessitent plus de conserver l’intégralité de la charge utile en mémoire sans casser les uploads existants.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- ios, objective-c
- Domaine
- mobile
- Type d'issue
- Fonctionnalité
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- À clarifier
- Accessibilité débutants
- 25/100