Investigate memory management issue with HttpRequest URLs in http_client.zig

Offen

#11 erstellt am 19.6.2025 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 4/5
Geschätzter Aufwand: 3-5 Tage
Anfängerfreundlichkeit: 35/100

Issue-Typ: Refactoring
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: zig
Bereich: testing

Rechercherichtung

Beginne mit http_client.zig und verfolge den Besitz der HttpRequest-URL durch die Initialisierung und deinit. Untersuche anschließend den referenzierten Testaufbau. Ermittle, wie globale Strings behandelt werden und ob der bestehende Duplizierungs-Workaround noch benötigt wird. Als abgeschlossen gilt die Änderung, wenn die Bereinigung der Anfrage nicht mehr versucht, nicht besessene globale Strings freizugeben, und die betroffenen Tests erfolgreich sind.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

bug

Review and refactor how memory is managed for HttpRequest URLs, particularly in test scenarios where the deinit method tries to free global strings. Consider alternative approaches for test setups or memory management.

Reference:

// TODO: The request de-allocates the memory used by the url in the deinit method.
// Because of this, with this test, we have to dupe it or else it cant deinit a global string.
// Needing to do this is a sign that something is off here, but also newing up requests directly
// isn't really intended either. Either way, need to look into this further.

Vorherrschende Sprache: Zig
Sterne: 11
Forks: 1
PR-Merge-Kennzahlen: Keine gemergten PRs in 30 T.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus bradcypert/httpspec

Diagnostic reporting should use some fancy color formatting Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 42/100

bradcypert/httpspec#22 · 1 Kommentar · Erstellt am 20.8.2025
Support named requests in HttpRequest (parser.zig) Offen

enhancement

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100

bradcypert/httpspec#12 · Erstellt am 19.6.2025

Alle Issues in bradcypert/httpspec