drogonframework/drogon

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

Open

#2 139 ouverte le 24 août 2024

Voir sur GitHub
 (5 commentaires) (1 réaction) (0 assignés)C++ (1 014 forks)batch import
enhancementgood first issue

Métriques du dépôt

Stars
 (10 462 stars)
Métriques de merge PR
 (Merge moyen 2j 2h) (7 PRs mergées en 30 j)

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.

Guide contributeur

Direction de recherche
Examinez la classe HttpResponse dans le code source de Drogon, en particulier les surcharges de la méthode setBody. Regardez également la méthode send du flux asynchrone. L'issue demande l'ajout d'une nouvelle surcharge acceptant char* et size t. Examinez les surcharges existantes pour garantir une conception d'API correcte. Aucune PR liée ou fichiers spécifiques mentionnés.
Stack technique
cpp
Domaine
backendapi
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
basic C++familiarity with Drogon
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70