public/browser
1
0
Fork 0
mirror of https://github.com/lightpanda-io/browser.git synced 2026-03-22 04:34:44 +00:00
Code Issues Actions 5 Packages Projects Releases Wiki Activity

populate Response.statusText for network responses

Browse Source 
Fetch responses from the network now have their statusText set using
the HTTP status phrase (e.g. "OK", "Not Found"), matching the Fetch
spec and the existing XMLHttpRequest behavior.
This commit is contained in:
egrs
2026-02-17 15:49:20 +01:00
parent db4a97743f
commit 2aa09ae18d
2 changed files with 1 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
src/browser/webapi/net/Fetch.zig
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;

4
src/browser/webapi/net/Response.zig
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;
} }