[sr-dev] [kamailio/kamailio] Add iteration by all DNS NAPTR records (#2290)

[Semen Darienko]

Current stable release support using only one (first by ordering) NAPTR DNS record. 
I didn't find any ideas how to configure possibility for use all supported NAPTR / SRV records. This pull request resolve this issue.

General issue:
PreConditions:
# host -t naptr voip.somehost.com
voip.somehost.com has NAPTR record 1 0 "S" "SIPS+D2T" "" _sips._tcp.voip.somehost.com.
voip.somehost.com has NAPTR record 2 0 "S" "SIP+D2U" "" _sip._udp.voip.somehost.com.

# host -t srv _sips._tcp.voip.somehost.com
_sips._tcp.voip.somehost.com has SRV record 1 1 443 voip.somehost.com.
_sips._tcp.voip.somehost.com has SRV record 2 1 5061 192.168.1.50.
_sips._tcp.voip.somehost.com has SRV record 3 1 5061 10.100.1.20.

# host -t srv _sip._udp.voip.somehost.com
_sip._udp.voip.somehost.com has SRV record 1 1 5060 voip.somehost.com.
_sip._udp.voip.somehost.com has SRV record 2 1 5060 192.168.1.50.
_sip._udp.voip.somehost.com has SRV record 3 1 5060 10.100.1.20.

In my configuration of kamailio I don't see trying to use second by priority NAPTR record:
voip.somehost.com has NAPTR record 2 0 "S" "SIP+D2U" "" _sip._udp.voip.somehost.com.

Kindly ask to share your ideas regarding faced issue and my pull request.
Thank you.

You can view, comment on, or merge this pull request online at:

  https://github.com/kamailio/kamailio/pull/2290

-- Commit Summary --

  * Add using all NAPTR records

-- File Changes --

    M src/core/dns_cache.c (86)
    M src/core/resolve.c (1)
    M src/core/resolve.h (1)

-- Patch Links --

https://github.com/kamailio/kamailio/pull/2290.patch
https://github.com/kamailio/kamailio/pull/2290.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/2290
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.kamailio.org/pipermail/sr-dev/attachments/20200417/11349e90/attachment.html>