@herlesupreeth commented on this pull request.
> + 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 or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3914#pullrequestreview-2176473762
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3914/review/2176473...@github.com>_______________________________________________
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org
