For upload, consider using [NSMutableURLRequest setHTTPBodyStream] instead of setHTTPBody
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Facilidade para iniciantes
- 25/100
- Tipo de issue
- Funcionalidade
- Clareza
- Precisa de esclarecimento
- Status de atividade
- Estagnada
- Stack de tecnologia
- ios, objective-c
- Domínio
- mobile
Direção de pesquisa
Comece localizando a implementação da requisição de upload do iOS que atualmente atribui os dados de upload com setHTTPBody e, em seguida, inspecione como os uploads de arquivos e vídeos são construídos. Compare o fluxo de requisição existente com setHTTPBodyStream de NSMutableURLRequest e determine como funcionam a conclusão e o tratamento de erros. O trabalho estará concluído quando uploads grandes não exigirem mais que todo o payload fique na memória sem interromper os uploads existentes.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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.
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
- Linguagem predominante
- Swift
- Estrelas
- 169
- Forks
- 107
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Sem guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de cloudinary/cloudinary_ios
-
Memory Leaks and Retain Cycles while using Upload with Cloudinary SDKTalvez livre de novo @adimiz1 assumiu há 713 dias e não há nenhum pull request aberto. Aberta
cloudinary/cloudinary_ios#441 · 1 comentário · 1 responsável ·
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 20/100
cloudinary/cloudinary_ios#439 ·
-
Add MD5-Header option to CLDUploadRequestParamsTalvez livre de novo @adimiz1 assumiu há 1458 dias e não há nenhum pull request aberto. Aberta
cloudinary/cloudinary_ios#390 · 1 comentário · 1 responsável ·
-
Unable to upload photo, uploadPrefix, unrecognized selectorTalvez livre de novo @adimiz1 assumiu há 1266 dias e não há nenhum pull request aberto. Abertabug
cloudinary/cloudinary_ios#369 · 2 comentários · 1 responsável ·
-
Does not compile for tvOSTalvez livre de novo @GiniAppsPartners assumiu há 2026 dias e não há nenhum pull request aberto. Aberta
cloudinary/cloudinary_ios#337 · 1 comentário · 2 responsáveis ·
Todas as issues de cloudinary/cloudinary_ios
Issues semelhantes
-
cua-driver diagnose: one-line help says "install and permission report", which is macOS-specificAberta
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 78/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
Mantenedores costumam responder em até 1 dia
-
easy
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 62/100
Mantenedores costumam responder em até 5 dias
-
15-minutes 5-minutes algorithm any-language beginner-friendly c cpp csharp easy go good first issue hacktoberfest help wanted java javascript kotlin python rust swift typescript
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 68/100
Huerte/algo-solutions-python#260 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100