drogonframework/drogon

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

Open

#2.139 geöffnet am 24. Aug. 2024

Auf GitHub ansehen
 (5 Kommentare) (1 Reaktion) (0 zugewiesene Personen)C++ (1.014 Forks)batch import
enhancementgood first issue

Repository-Metriken

Stars
 (10.462 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 2T 2h) (7 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuche die HttpResponse Klasse im Drogon Quellcode, insbesondere die Überladungen der setBody Methode. Sieh dir auch die send Methode des asynchronen Streams an. Das Issue fordert das Hinzufügen einer neuen Überladung, die char* und size t akzeptiert. Überprüfe die vorhandenen Überladungen, um ein korrektes API Design sicherzustellen. Keine verknüpften PRs oder spezifische Dateien erwähnt.
Tech Stack
cpp
Domain
backendapi
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
basic C++familiarity with Drogon
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70