http/client: Throw error early if .path given and non AF_UNIX family

Author: daurnimator

http/client.lua

@@ -127,6 +127,15 @@ local function connect(options, timeout)
 		family = cs.AF_UNSPEC
 	end
 
+	local path = options.path
+	if path then
+		if family == cs.AF_UNSPEC then
+			family = cs.AF_UNIX
+		elseif family ~= cs.AF_UNIX then
+			error("cannot use .path with non-unix address family")
+		end
+	end
+
 	local bind = options.bind
 	if bind ~= nil then
 		assert(type(bind) == "string")
@@ -157,7 +166,6 @@ local function connect(options, timeout)
 		}
 	end
 
-	local path = options.path
 	local host = options.host
 	if not path and not http_util.is_ip(host) then
 		local dns_resolver = options.dns_resolver or cqueues_dns.getpool()

spec/client_spec.lua

@@ -15,6 +15,11 @@ describe("http.client module", function()
 	local openssl_pkey = require "openssl.pkey"
 	local openssl_ctx = require "openssl.ssl.context"
 	local openssl_x509 = require "openssl.x509"
+	it("throws error on invalid family+path combination", function()
+		assert.has.errors(function()
+			client.connect{family = cs.AF_INET, path = "/somepath"}
+		end)
+	end)
 	it("throws error on invalid family+bind combination", function()
 		assert.has.errors(function()
 			client.connect{family = cs.AF_INET, host = "example.com", port = 80, bind = "::1" }