Merge pull request #1573 from egrs/response-statustext

Populate Response.statusText for network responses
This commit is contained in:
Karl Seguin
2026-02-18 07:28:55 +08:00
committed by GitHub
2 changed files with 1 additions and 4 deletions

View File

@@ -117,6 +117,7 @@ fn httpHeaderDoneCallback(transfer: *Http.Transfer) !bool {
} }
res._status = header.status; res._status = header.status;
res._status_text = std.http.Status.phrase(@enumFromInt(header.status)) orelse "";
res._url = try arena.dupeZ(u8, std.mem.span(header.url)); res._url = try arena.dupeZ(u8, std.mem.span(header.url));
res._is_redirected = header.redirect_count > 0; res._is_redirected = header.redirect_count > 0;

View File

@@ -95,10 +95,6 @@ pub fn getStatus(self: *const Response) u16 {
} }
pub fn getStatusText(self: *const Response) []const u8 { pub fn getStatusText(self: *const Response) []const u8 {
// @TODO
// This property is meant to actually capture the response status text, not
// just return the text representation of self._status. If we do,
// new Response(null, {status: 200}).statusText, we should get empty string.
return self._status_text; return self._status_text;
} }