@herlesupreeth commented on this pull request.


In src/modules/cdp/routing.c:

> +	avp = AAAFindMatchingAVP(m, 0, AVP_Destination_Host, 0, AAA_FORWARD_SEARCH);
+	if(avp)
+		destination_host = avp->data;
+
+	if(destination_host.len) {
+		/* There is a destination host present in the message try and route directly there */
+		p = get_peer_by_fqdn(&destination_host);
+		if(p && (p->state == I_Open || p->state == R_Open)
+				&& peer_handles_application(p, app_id, vendor_id)) {
+			p->last_selected = time(NULL);
+			return p;
+		}
+		/* the destination host peer is not connected at the moment, try a normal route then */
+	}
+
+	if(!config->r_table) {

cant we remove this check here as its already done in get_peer_by_fqdn ??


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <kamailio/kamailio/pull/3914/review/2176473762@github.com>