authorgravatar for me@eimmer.meElijah M. Immer <me@eimmer.me> 2025-03-14 13:20:42-07:00
committergravatar for alex@alexrp.comAlex Rønne Petersen <alex@alexrp.com> 2025-03-14 21:20:55+01:00
logf79dacbfc4876c2799c7a2dcd2b29af71d43d006
treec25d331fe78eef60bf17f73445a56006b63c9b3c
parentdc75a64c468c99f801af47b7a0dfc76121a46e8b
signaturebadge-check Signed by SSH key SHA256:7B/LJ7bpR1eX8aCXSr4mtd5M45VMPKcx9zY8e95b5QM

lib/std/http/Client.zig: Ignore empty proxy environment variables (#23223)

This fixes #21032 by ignoring proxy environment variables that are empty.

1 files changed, 5 insertions(+), 1 deletions(-)

lib/std/http/Client.zig+5-1
...@@ -1241,10 +1241,14 @@ pub fn initDefaultProxies(client: *Client, arena: Allocator) !void {...@@ -1241,10 +1241,14 @@ pub fn initDefaultProxies(client: *Client, arena: Allocator) !void {
12411241
1242fn createProxyFromEnvVar(arena: Allocator, env_var_names: []const []const u8) !?*Proxy {1242fn createProxyFromEnvVar(arena: Allocator, env_var_names: []const []const u8) !?*Proxy {
1243 const content = for (env_var_names) |name| {1243 const content = for (env_var_names) |name| {
1244 break std.process.getEnvVarOwned(arena, name) catch |err| switch (err) {1244 const content = std.process.getEnvVarOwned(arena, name) catch |err| switch (err) {
1245 error.EnvironmentVariableNotFound => continue,1245 error.EnvironmentVariableNotFound => continue,
1246 else => |e| return e,1246 else => |e| return e,
1247 };1247 };
1248
1249 if (content.len == 0) continue;
1250
1251 break content;
1248 } else return null;1252 } else return null;
12491253
1250 const uri = Uri.parse(content) catch try Uri.parseAfterScheme("http", content);1254 const uri = Uri.parse(content) catch try Uri.parseAfterScheme("http", content);