mirrors/bun.sh
1
0
Fork 0
mirror of https://github.com/oven-sh/bun synced 2026-02-09 10:28:47 +00:00
Code Issues Packages Projects Releases Wiki Activity

In http client, use .toOwnedSlice() instead of potentially re-using the WTFString here

Browse Source
This commit is contained in:
Jarred Sumner
2023-09-20 23:40:42 -07:00
parent b00588e98c
commit e160223627
1 changed files with 7 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
src/http_client_async.zig
View File

@@ -3488,11 +3488,14 @@ pub fn handleResponseMetadata(
);
defer new_url_.deref();
if (new_url_.utf8ByteLength() > url_buf.data.len) {
return error.RedirectURLTooLong;
if (new_url_.isEmpty()) {
return error.InvalidRedirectURL;
}
const new_url = new_url_.toUTF8(fba.allocator());
this.url = URL.parse(new_url.slice());
const new_url = new_url_.toOwnedSlice(fba.allocator()) catch {
return error.RedirectURLTooLong;
};
this.url = URL.parse(new_url);
is_same_origin = strings.eqlCaseInsensitiveASCII(strings.withoutTrailingSlash(this.url.origin), strings.withoutTrailingSlash(original_url.origin), true);
deferred_redirect.* = this.redirect;