drogonframework/drogon

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

Open

#2139 aperta il 24 ago 2024

Vedi su GitHub
 (5 commenti) (1 reazione) (0 assegnatari)C++ (1014 fork)batch import
enhancementgood first issue

Metriche repository

Star
 (10.462 star)
Metriche merge PR
 (Merge medio 2g 2h) (7 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Analizza la classe HttpResponse nel codice sorgente di Drogon, in particolare gli overload del metodo setBody. Esamina anche il metodo send dello stream asincrono. La richiesta del problema è di aggiungere un nuovo overload che accetti char* e size t. Esamina gli overload esistenti per garantire un corretto design dell'API. Nessun PR collegato o file specifico menzionato.
Tech stack
cpp
Dominio
backendapi
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
basic C++familiarity with Drogon
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70