certmanager: Adjust paths of SSL key/certs to be relative to the config file, fixes #147

Tue, 13 Jul 2010 13:56:14 +0100

author
Matthew Wild <mwild1@gmail.com>
date
Tue, 13 Jul 2010 13:56:14 +0100
changeset 3355
9bb2da325d4d
parent 3354
b4ebda3fd6e9
child 3356
cf6be653c619

certmanager: Adjust paths of SSL key/certs to be relative to the config file, fixes #147

core/certmanager.lua file | annotate | diff | comparison | revisions
--- a/core/certmanager.lua	Tue Jul 13 13:55:18 2010 +0100
+++ b/core/certmanager.lua	Tue Jul 13 13:56:14 2010 +0100
@@ -6,53 +6,63 @@
 local setmetatable, tostring = setmetatable, tostring;
 
 local prosody = prosody;
+local resolve_path = prosody.resolve_relative_path;
 
 module "certmanager"
 
--- These are the defaults if not overridden in the config
-local default_ssl_ctx = { mode = "client", protocol = "sslv23", capath = "/etc/ssl/certs", verify = "none", options = "no_sslv2"; };
-local default_ssl_ctx_in = { mode = "server", protocol = "sslv23", capath = "/etc/ssl/certs", verify = "none", options = "no_sslv2"; };
-
-local default_ssl_ctx_mt = { __index = default_ssl_ctx };
-local default_ssl_ctx_in_mt = { __index = default_ssl_ctx_in };
-
 -- Global SSL options if not overridden per-host
 local default_ssl_config = configmanager.get("*", "core", "ssl");
 
 function create_context(host, mode, config)
-	local ssl_config = config and config.core.ssl or default_ssl_config;
-	if ssl and ssl_config then
-		local ctx, err = ssl_newcontext(setmetatable(ssl_config, mode == "client" and default_ssl_ctx_mt or default_ssl_ctx_in_mt));
-		if not ctx then
-			err = err or "invalid ssl config"
-			local file = err:match("^error loading (.-) %(");
-			if file then
-				if file == "private key" then
-					file = ssl_config.key or "your private key";
-				elseif file == "certificate" then
-					file = ssl_config.certificate or "your certificate file";
-				end
-				local reason = err:match("%((.+)%)$") or "some reason";
-				if reason == "Permission denied" then
-					reason = "Check that the permissions allow Prosody to read this file.";
-				elseif reason == "No such file or directory" then
-					reason = "Check that the path is correct, and the file exists.";
-				elseif reason == "system lib" then
-					reason = "Previous error (see logs), or other system error.";
-				elseif reason == "(null)" or not reason then
-					reason = "Check that the file exists and the permissions are correct";
-				else
-					reason = "Reason: "..tostring(reason):lower();
-				end
-				log("error", "SSL/TLS: Failed to load %s: %s", file, reason);
+	if not ssl then return nil; end
+	
+	local user_ssl_config = config and config.core.ssl or default_ssl_config;
+	
+	local ssl_config = {
+		mode = mode;
+		protocol = user_ssl_config.protocol or "sslv23";
+		key = resolve_path(user_ssl_config.key);
+		password = user_ssl_config.password;
+		certificate = resolve_path(user_ssl_config.certificate);
+		capath = resolve_path(user_ssl_config.capath or default_capath);
+		cafile = resolve_path(user_ssl_config.cafile);
+		verify = user_ssl_config.verify or "none";
+		options = user_ssl_config.options or "no_sslv2";
+		ciphers = user_ssl_config.ciphers;
+		depth = user_ssl_config.depth;
+	};
+
+	log("warn", "keyfile: %q", ssl_config.key);
+
+	local ctx, err = ssl_newcontext(ssl_config);
+	if not ctx then
+		err = err or "invalid ssl config"
+		local file = err:match("^error loading (.-) %(");
+		if file then
+			if file == "private key" then
+				file = ssl_config.key or "your private key";
+			elseif file == "certificate" then
+				file = ssl_config.certificate or "your certificate file";
+			end
+			local reason = err:match("%((.+)%)$") or "some reason";
+			if reason == "Permission denied" then
+				reason = "Check that the permissions allow Prosody to read this file.";
+			elseif reason == "No such file or directory" then
+				reason = "Check that the path is correct, and the file exists.";
+			elseif reason == "system lib" then
+				reason = "Previous error (see logs), or other system error.";
+			elseif reason == "(null)" or not reason then
+				reason = "Check that the file exists and the permissions are correct";
 			else
-				log("error", "SSL/TLS: Error initialising for host %s: %s", host, err );
+				reason = "Reason: "..tostring(reason):lower();
 			end
-			ssl = false
-        	end
-        	return ctx, err;
-	end
-	return nil;
+			log("error", "SSL/TLS: Failed to load %s: %s", file, reason);
+		else
+			log("error", "SSL/TLS: Error initialising for host %s: %s", host, err );
+		end
+		ssl = false
+        end
+        return ctx, err;
 end
 
 function reload_ssl_config()

mercurial