cloudinary / cloudinary/cloudinary_ios

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

Ouverte
#37 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
enhancement
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

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.