[Asterisk-code-review] res pjsip endpoint identifier ip: Add support for SRV lookups. (asterisk[13])

[George Joseph]

George Joseph has posted comments on this change. ( https://gerrit.asterisk.org/4697 )

Change subject: res_pjsip_endpoint_identifier_ip: Add support for SRV lookups.
......................................................................


Patch Set 3:

> That incurs a further user burden - they have to investigate and
 > understand the topology of their ITSP. Specifically whether they
 > are using SRV records or not. Users generally don't care, they just
 > want it to work.

It's the not-generally cases I worry about like high-volume service providers.  How about having it on by default and having the option to disable it?

-- 
To view, visit https://gerrit.asterisk.org/4697
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I6b641e275bf96629320efa8b479737062aed82ac
Gerrit-PatchSet: 3
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-HasComments: No