configmanager: Error when a component and host clash hostnames

Sun, 14 Feb 2010 18:41:44 +0000

author
Matthew Wild <mwild1@gmail.com>
date
Sun, 14 Feb 2010 18:41:44 +0000
changeset 2637
82cfc1ec4a7c
parent 2636
d2805ad5b736
child 2638
115104acdd7b

configmanager: Error when a component and host clash hostnames

core/configmanager.lua file | annotate | diff | comparison | revisions
--- a/core/configmanager.lua	Sun Feb 14 10:00:39 2010 +0500
+++ b/core/configmanager.lua	Sun Feb 14 18:41:44 2010 +0000
@@ -9,8 +9,11 @@
 
 
 local _G = _G;
-local 	setmetatable, loadfile, pcall, rawget, rawset, io, error, dofile, type, pairs, table =
-		setmetatable, loadfile, pcall, rawget, rawset, io, error, dofile, type, pairs, table;
+local 	setmetatable, loadfile, pcall, rawget, rawset, io, error, dofile, type, pairs, table, format =
+		setmetatable, loadfile, pcall, rawget, rawset, io, error, dofile, type, pairs, table, string.format;
+
+
+local trb = debug.traceback
 
 local eventmanager = require "core.eventmanager";
 
@@ -124,6 +127,10 @@
 		
 		rawset(env, "__currenthost", "*") -- Default is global
 		function env.Host(name)
+			if rawget(config, name) and rawget(config[name].core, "component_module") then
+				error(format("Host %q clashes with previously defined %s Component %q, for services use a sub-domain like conference.%s",
+					name, config[name].core.component_module:gsub("^%a+$", { component = "external", muc = "MUC"}), name, name), 0);
+			end
 			rawset(env, "__currenthost", name);
 			-- Needs at least one setting to logically exist :)
 			set(name or "*", "core", "defined", true);
@@ -131,6 +138,10 @@
 		env.host = env.Host;
 		
 		function env.Component(name)
+			if rawget(config, name) and rawget(config[name].core, "defined") and not rawget(config[name].core, "component_module") then
+				error(format("Component %q clashes with previously defined Host %q, for services use a sub-domain like conference.%s",
+					name, name, name), 0);
+			end
 			set(name, "core", "component_module", "component");
 			-- Don't load the global modules by default
 			set(name, "core", "load_global_modules", false);

mercurial