[Freeswitch-trunk] [commit] r12688 - freeswitch/trunk/src/mod/applications/mod_lcr

FreeSWITCH SVN rupa at freeswitch.org
Fri Mar 20 13:15:39 PDT 2009


Author: rupa
Date: Fri Mar 20 15:15:39 2009
New Revision: 12688

Log:
Add support for specifying codec in carrier_gateway table
Add ability to test ${} queries even when no session is
 available (abuse events for this)
Test sql at startup, refuse to load invalid profile


Modified:
   freeswitch/trunk/src/mod/applications/mod_lcr/mod_lcr.c

Modified: freeswitch/trunk/src/mod/applications/mod_lcr/mod_lcr.c
==============================================================================
--- freeswitch/trunk/src/mod/applications/mod_lcr/mod_lcr.c	(original)
+++ freeswitch/trunk/src/mod/applications/mod_lcr/mod_lcr.c	Fri Mar 20 15:15:39 2009
@@ -47,8 +47,10 @@
 #define LCR_TSTRIP_PLACE 6
 #define LCR_PREFIX_PLACE 7
 #define LCR_SUFFIX_PLACE 8
+#define LCR_CODEC_PLACE 9
 
-#define LCR_QUERY_COLS 9
+#define LCR_QUERY_COLS_REQUIRED 9
+#define LCR_QUERY_COLS 10
 
 #define LCR_DIALSTRING_PLACE 3
 #define LCR_HEADERS_COUNT 4
@@ -84,6 +86,7 @@
 	size_t lstrip;
 	size_t tstrip;
 	size_t digit_len;
+	char *codec;
 	struct lcr_obj *prev;
 	struct lcr_obj *next;
 };
@@ -126,6 +129,7 @@
 	char *lookup_number;
 	profile_t *profile;
 	switch_core_session_t *session;
+	switch_event_t *event;
 };
 typedef struct callback_obj callback_t;
 
@@ -153,6 +157,7 @@
 	char *data = NULL;
 	char *destination_number = NULL;
 	char *orig_destination_number = NULL; 
+	char *codec = NULL;
 
 	orig_destination_number = destination_number = switch_core_strdup(pool, dialed_number);
 
@@ -166,8 +171,14 @@
 		destination_number += lstrip;
 	}
 	
-	data = switch_core_sprintf(pool, "[lcr_carrier=%s,lcr_rate=%s]%s%s%s%s%s"
+	codec = "";
+	if (!switch_strlen_zero(cur_route->codec)) {
+		codec = switch_core_sprintf(pool, ",absolute_codec_string=%s", cur_route->codec);
+	}
+	
+	data = switch_core_sprintf(pool, "[lcr_carrier=%s,lcr_rate=%s%s]%s%s%s%s%s"
 								, cur_route->carrier_name, cur_route->rate_str
+								, codec
 								, cur_route->gw_prefix, cur_route->prefix
 								, destination_number, cur_route->suffix, cur_route->gw_suffix);
 			
@@ -243,24 +254,29 @@
 	return SWITCH_STATUS_SUCCESS;
 }
 
-/* try each type of random until we suceed */
-static switch_bool_t set_db_random()
+static switch_bool_t db_check(char *sql)
 {
-	char *sql = NULL;
 	if (globals.odbc_dsn) {
-		sql = "SELECT rand()";
-		if (switch_odbc_handle_exec(globals.master_odbc, sql, NULL)
-				== SWITCH_ODBC_SUCCESS) {
-			db_random = "rand()";
-			return SWITCH_TRUE;
-		}
-		sql = "SELECT random()";
-		if (switch_odbc_handle_exec(globals.master_odbc, sql, NULL)
-				== SWITCH_ODBC_SUCCESS) {
-			db_random = "random()";
+		if (switch_odbc_handle_exec(globals.master_odbc, sql, NULL) == SWITCH_ODBC_SUCCESS) {
 			return SWITCH_TRUE;
 		}
 	}
+	
+	return SWITCH_FALSE;
+}
+
+/* try each type of random until we suceed */
+static switch_bool_t set_db_random()
+{
+	if (db_check("SELECT rand();") == SWITCH_TRUE) {
+		db_random = "rand()";
+		return SWITCH_TRUE;
+	}
+	if(db_check("SELECT random();") == SWITCH_TRUE) {
+		db_random = "random()";
+		return SWITCH_TRUE;
+	}
+	
 	return SWITCH_FALSE;
 }
 
@@ -348,6 +364,10 @@
 			*/
 			newSQL = switch_channel_expand_variables(channel, 
 														tmpSQL ? tmpSQL : custom_sql);
+		} else if (cb_struct->event) {
+			/* use event system to expand vars */
+			newSQL = switch_event_expand_headers(cb_struct->event, tmpSQL ? tmpSQL : custom_sql);
+			
 		} else {
 			switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_CRIT,
 								"mod_lcr called without a valid session while using a custom_sql that has channel variables.\n");
@@ -392,9 +412,9 @@
 	switch_memory_pool_t *pool = cbt->pool;
 	
 	
-	if (argc != LCR_QUERY_COLS) {
+	if (argc < LCR_QUERY_COLS_REQUIRED) {
 		switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_CRIT,
-							"Unexpected number of columns returned for SQL.  Returned columns are: %d. "
+							"Unexpected number of columns returned for SQL.  Returned column count: %d. "
 							"If using a custom sql for this profile, verify it is correct.  Otherwise file a bug report.\n",
 							argc);
 		return SWITCH_STATUS_GENERR;
@@ -422,6 +442,9 @@
 	additional->gw_suffix = switch_core_strdup(pool, switch_str_nil(argv[LCR_GW_SUFFIX_PLACE]));
 	additional->lstrip = atoi(switch_str_nil(argv[LCR_LSTRIP_PLACE]));
 	additional->tstrip = atoi(switch_str_nil(argv[LCR_TSTRIP_PLACE]));
+	if (argc > LCR_CODEC_PLACE) {
+		additional->codec = switch_core_strdup(pool, switch_str_nil(argv[LCR_CODEC_PLACE]));
+	}
 	additional->dialstring = get_bridge_data(pool, cbt->lookup_number, additional);
 
 	if (cbt->head == NULL) {
@@ -497,7 +520,6 @@
 
 switch_status_t lcr_do_lookup(callback_t *cb_struct, char *digits)
 {
-	/* instantiate the object/struct we defined earlier */
 	switch_stream_handle_t sql_stream = { 0 };
 	char *digits_copy;
 	char *digits_expanded;
@@ -517,9 +539,6 @@
 		return SWITCH_STATUS_FALSE;
 	}
 	
-	/* SWITCH_STANDARD_STREAM doesn't use pools.  but we only have to free sql_stream.data */
-	SWITCH_STANDARD_STREAM(sql_stream);
-	
 	digits_expanded = expand_digits(cb_struct->pool, digits_copy, cb_struct->profile->quote_in_list);
 	
 	/* set some channel vars if we have a session */
@@ -531,39 +550,26 @@
 			switch_channel_set_variable_var_check(channel, "lcr_query_expanded_digits", digits_expanded, SWITCH_FALSE);
 		}
 	}
+	if (cb_struct->event) {
+		switch_event_add_header_string(cb_struct->event, SWITCH_STACK_BOTTOM, "lcr_query_digits", digits_copy);
+		id_str = switch_core_sprintf(cb_struct->pool, "%d", cb_struct->profile->id);
+		switch_event_add_header_string(cb_struct->event, SWITCH_STACK_BOTTOM, "lcr_query_profile", id_str);
+		switch_event_add_header_string(cb_struct->event, SWITCH_STACK_BOTTOM, "lcr_query_expanded_digits", digits_expanded);
+	}
 
 	/* set up the query to be executed */
-	if (switch_strlen_zero(profile->custom_sql)) {
-		sql_stream.write_function(&sql_stream, 
-								  "SELECT l.digits, c.carrier_name, l.rate, cg.prefix AS gw_prefix, cg.suffix AS gw_suffix, l.lead_strip, l.trail_strip, l.prefix, l.suffix "
-								  );
-		sql_stream.write_function(&sql_stream, "FROM lcr l JOIN carriers c ON l.carrier_id=c.id JOIN carrier_gateway cg ON c.id=cg.carrier_id WHERE c.enabled = '1' AND cg.enabled = '1' AND l.enabled = '1' AND digits IN (");
-		sql_stream.write_function(&sql_stream, "%s", digits_expanded);
-		sql_stream.write_function(&sql_stream, ") AND CURRENT_TIMESTAMP BETWEEN date_start AND date_end ");
-		if (profile->id > 0) {
-			sql_stream.write_function(&sql_stream, "AND lcr_profile=%d ", profile->id);
-		}
-		sql_stream.write_function(&sql_stream, "ORDER BY %s%s digits DESC%s", 
-												profile->pre_order,
-												switch_strlen_zero(profile->pre_order)? "" : ",",
-												profile->order_by);
-		if (db_random) {
-			sql_stream.write_function(&sql_stream, ", %s", db_random);
-		}
-		sql_stream.write_function(&sql_stream, ";");
-	} else {
-		char *safe_sql;
+	char *safe_sql;
 
-		/* format the custom_sql */
-		safe_sql = format_custom_sql(profile->custom_sql, cb_struct, digits_copy);
-		if (!safe_sql) {
-			return SWITCH_STATUS_GENERR;
-		}
-		sql_stream.write_function(&sql_stream, safe_sql);
-		if (safe_sql != profile->custom_sql) {
-			/* channel_expand_variables returned the same string to us, no need to free */
-			switch_safe_free(safe_sql);
-		}
+	/* format the custom_sql */
+	safe_sql = format_custom_sql(profile->custom_sql, cb_struct, digits_copy);
+	if (!safe_sql) {
+		return SWITCH_STATUS_GENERR;
+	}
+	SWITCH_STANDARD_STREAM(sql_stream);
+	sql_stream.write_function(&sql_stream, safe_sql);
+	if (safe_sql != profile->custom_sql) {
+		/* channel_expand_variables returned the same string to us, no need to free */
+		switch_safe_free(safe_sql);
 	}
 	
 	switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_DEBUG, "SQL: %s\n", (char *)sql_stream.data);    
@@ -580,9 +586,30 @@
 	}
 }
 
+switch_status_t test_profile(char *lcr_profile)
+{
+	callback_t routes = { 0 };
+	switch_memory_pool_t *pool = NULL;
+	switch_event_t *event = NULL;
+
+	switch_core_new_memory_pool(&pool);
+	switch_event_create(&event, SWITCH_EVENT_MESSAGE);
+	routes.event = event;
+	routes.pool = pool;
+	
+	if (!(routes.profile = locate_profile(lcr_profile))) {
+		switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_ERROR, "Unknown profile: %s\n", lcr_profile);
+		return SWITCH_FALSE;
+	}
+	
+	routes.lookup_number = "15555551212";
+	return lcr_do_lookup(&routes, routes.lookup_number);
+}
+
 static switch_status_t lcr_load_config()
 {
 	char *cf = "lcr.conf";
+	switch_stream_handle_t sql_stream = { 0 };
 	switch_xml_t cfg, xml, settings, param, x_profile, x_profiles;
 	switch_status_t status = SWITCH_STATUS_SUCCESS;
 	char *odbc_user = NULL;
@@ -631,6 +658,12 @@
 		}
 	}
 	
+	if (set_db_random() == SWITCH_TRUE) {
+		switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_INFO, "Database RANDOM function set to %s\n", db_random);
+	} else {
+		switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_ERROR, "Unable to determine database RANDOM function\n");
+	};
+
 	switch_core_hash_init(&globals.profile_hash, globals.pool);
 	if ((x_profiles = switch_xml_child(cfg, "profiles"))) {
 		for (x_profile = switch_xml_child(x_profiles, "profile"); x_profile; x_profile = x_profile->next) {
@@ -722,16 +755,45 @@
 					profile->id = (uint16_t)atoi(id_s);
 				}
 				
-				if (!switch_strlen_zero(custom_sql)) {
-					if (switch_string_var_check_const(custom_sql)) {
-						profile->custom_sql_has_vars = SWITCH_TRUE;
+				/* SWITCH_STANDARD_STREAM doesn't use pools.  but we only have to free sql_stream.data */
+				SWITCH_STANDARD_STREAM(sql_stream);
+				if (switch_strlen_zero(custom_sql)) {
+					/* use default sql */
+					sql_stream.write_function(&sql_stream, 
+											  "SELECT l.digits, c.carrier_name, l.rate, cg.prefix AS gw_prefix, cg.suffix AS gw_suffix, l.lead_strip, l.trail_strip, l.prefix, l.suffix "
+											  );
+					if (db_check("SELECT codec from carrier_gateway limit 1") == SWITCH_TRUE) {
+						sql_stream.write_function(&sql_stream, ", cg.codec ");
+						switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_DEBUG, "codec field defined.\n");
+					} else {
+						switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_WARNING, "codec field not defined, please update your lcr database schema.\n");
+					}
+					sql_stream.write_function(&sql_stream, "FROM lcr l JOIN carriers c ON l.carrier_id=c.id JOIN carrier_gateway cg ON c.id=cg.carrier_id WHERE c.enabled = '1' AND cg.enabled = '1' AND l.enabled = '1' AND digits IN (");
+					sql_stream.write_function(&sql_stream, "${lcr_query_expanded_digits}");
+					sql_stream.write_function(&sql_stream, ") AND CURRENT_TIMESTAMP BETWEEN date_start AND date_end ");
+					if (profile->id > 0) {
+						sql_stream.write_function(&sql_stream, "AND lcr_profile=%d ", profile->id);
 					}
-					if (strstr(custom_sql, "%")) {
-						profile->custom_sql_has_percent = SWITCH_TRUE;
+					sql_stream.write_function(&sql_stream, "ORDER BY %s%s digits DESC%s", 
+															profile->pre_order,
+															switch_strlen_zero(profile->pre_order)? "" : ",",
+															profile->order_by);
+					if (db_random) {
+						sql_stream.write_function(&sql_stream, ", %s", db_random);
 					}
-					profile->custom_sql = switch_core_strdup(globals.pool, (char *)custom_sql);
+					sql_stream.write_function(&sql_stream, ";");
+					
+					custom_sql = sql_stream.data;
 				}
 				
+				if (switch_string_var_check_const(custom_sql)) {
+					profile->custom_sql_has_vars = SWITCH_TRUE;
+				}
+				if (strstr(custom_sql, "%")) {
+					profile->custom_sql_has_percent = SWITCH_TRUE;
+				}
+				profile->custom_sql = switch_core_strdup(globals.pool, (char *)custom_sql);
+				
 				if (!switch_strlen_zero(reorder_by_rate)) {
 					profile->reorder_by_rate = switch_true(reorder_by_rate);
 				}
@@ -742,14 +804,20 @@
 				
 				switch_core_hash_insert(globals.profile_hash, profile->name, profile);
 				switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_INFO, "Loaded lcr profile %s.\n", profile->name);
-				
-				if (!strcasecmp(profile->name, "default")) {
-					globals.default_profile = profile;
-					switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_INFO, "Setting user defined default profile: %s.\n", profile->name);
+				/* test the profile */
+				if (test_profile(profile->name) != SWITCH_TRUE) {
+					if (!strcasecmp(profile->name, "default")) {
+						globals.default_profile = profile;
+						switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_INFO, "Setting user defined default profile: %s.\n", profile->name);
+					}
+				} else {
+					switch_core_hash_delete(globals.profile_hash, profile->name);
 				}
+				
 			}
 			switch_safe_free(order_by.data);
 			switch_safe_free(pre_order.data);
+			switch_safe_free(sql_stream.data);
 		}
 	} else {
 		switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_INFO, "No lcr profiles defined.\n");
@@ -779,12 +847,15 @@
 	lcr_route cur_route = { 0 };
 	char *lcr_profile = NULL;
 	switch_memory_pool_t *pool = NULL;
+	switch_event_t *event = NULL;
 
 	if (session) {
 		pool = switch_core_session_get_pool(session);
 		routes.session = session;
 	} else {
 		switch_core_new_memory_pool(&pool);
+		switch_event_create(&event, SWITCH_EVENT_MESSAGE);
+		routes.event = event;
 	}
 	routes.pool = pool;
 	if (!(routes.profile = locate_profile(lcr_profile))) {
@@ -850,6 +921,7 @@
 	callback_t routes = { 0 };
 	lcr_route cur_route = { 0 };
 	switch_memory_pool_t *pool;
+	switch_event_t *event;
 
 	if (!(mydata = switch_core_session_strdup(session, data))) {
 		return;
@@ -860,6 +932,8 @@
 		routes.session = session;
 	} else {
 		switch_core_new_memory_pool(&pool);
+		switch_event_create(&event, SWITCH_EVENT_MESSAGE);
+		routes.event = event;
 	}
 	routes.pool = pool;
 
@@ -913,6 +987,7 @@
 	max_obj_t maximum_lengths = { 0 };
 	callback_t cb_struct = { 0 };
 	switch_memory_pool_t *pool;
+	switch_event_t *event;
 	switch_status_t lookup_status = SWITCH_STATUS_SUCCESS;
 
 	if (switch_strlen_zero(cmd)) {
@@ -924,6 +999,8 @@
 		cb_struct.session = session;
 	} else {
 		switch_core_new_memory_pool(&pool);
+		switch_event_create(&event, SWITCH_EVENT_MESSAGE);
+		cb_struct.event = event;
 	}
 	cb_struct.pool = pool;
 	
@@ -1090,11 +1167,6 @@
 
 	globals.pool = pool;
 
-	if (lcr_load_config() != SWITCH_STATUS_SUCCESS) {
-		switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_ERROR, "Unable to load lcr config file\n");
-		return SWITCH_STATUS_FALSE;
-	}
-
 	if (switch_mutex_init(&globals.mutex, SWITCH_MUTEX_NESTED, globals.pool) != SWITCH_STATUS_SUCCESS) {
 		switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_ERROR, "failed to initialize mutex\n");
 	}
@@ -1102,12 +1174,10 @@
 		switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_ERROR, "failed to initialize db_mutex\n");
 	}
 
-	
-	if (set_db_random() == SWITCH_TRUE) {
-		switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_INFO, "Database RANDOM function set to %s\n", db_random);
-	} else {
-		switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_ERROR, "Unable to determine database RANDOM function\n");
-	};
+	if (lcr_load_config() != SWITCH_STATUS_SUCCESS) {
+		switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_ERROR, "Unable to load lcr config file\n");
+		return SWITCH_STATUS_FALSE;
+	}
 
 	SWITCH_ADD_API(dialplan_lcr_api_interface, "lcr", "Least Cost Routing Module", dialplan_lcr_function, LCR_SYNTAX);
 	SWITCH_ADD_API(dialplan_lcr_api_admin_interface, "lcr_admin", "Least Cost Routing Module Admin", dialplan_lcr_admin_function, LCR_ADMIN_SYNTAX);



More information about the Freeswitch-trunk mailing list