drogonframework/drogon

Passing binary arrays directly to a resp->setBody()

Open

#2.139 aberto em 24 de ago. de 2024

Ver no GitHub
 (5 comments) (1 reaction) (0 assignees)C++ (1.014 forks)batch import
enhancementgood first issue

Métricas do repositório

Stars
 (10.462 stars)
Métricas de merge de PR
 (Mesclagem média 2d 2h) (7 fundiu PRs em 30d)

Description

It's disheartening to dance on eggshells when trying to pass a large binary array to drogon's setBody method. Nothing works. No shared_ptr, no string_view, nothing. The only way to pass a binary array right now seems to be using the list constructor of std::string by calling std::string fullResponse(char* myData, size_t myDataSize); and then calling std::move to hand it over to the setBody method, which involves a wholly unnecessary copy operation. Yes, compared to the compression, serialization, its not really THAT relevant. Nonetheless, what I would like is a method overload which accepts a char* and a size_t. The same goes for the send() method of the async stream.

Guia do colaborador

Direção de pesquisa
Investigue a classe HttpResponse no código fonte do Drogon, particularmente as sobrecargas do método setBody. Examine também o método send do fluxo assíncrono. A issue solicita a adição de uma nova sobrecarga que aceite char* e size t. Analise as sobrecargas existentes para garantir um design de API adequado. Nenhum PR vinculado ou arquivos específicos mencionados.
Pilha de tecnologia
cpp
Domain
backendapi
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
basic C++familiarity with Drogon
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70