[asterisk-bugs] [JIRA] (ASTERISK-26011) [patch]PJSIP: add "via_addr", "via_port", "call_id" to contacts

[Walter Doekes (JIRA)]

    [ https://issues.asterisk.org/jira/browse/ASTERISK-26011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=230548#comment-230548 ] 

Walter Doekes commented on ASTERISK-26011:
------------------------------------------

Hi. Thanks for the patches.
If you could you add the URL to the gerrit review in the "Reviewboard URL" field in the future, it helps. Thanks :)

I did not get a chance too look at your patch in gerrit at this point, because it is down (daily reload), but I wonder what the purpose of this patch is.

res_pjsip_nat rewrites the contact: ok, and then. When do we want the via_addr/via_port/callid, why do we want it? What would we do with it? Is it worth storing all of that for every registered peer always?



> [patch]PJSIP: add "via_addr", "via_port", "call_id" to contacts
> ---------------------------------------------------------------
>
>                 Key: ASTERISK-26011
>                 URL: https://issues.asterisk.org/jira/browse/ASTERISK-26011
>             Project: Asterisk
>          Issue Type: Improvement
>      Security Level: None
>          Components: Resources/res_pjsip, Resources/res_pjsip_registrar
>    Affects Versions: 13.9.0
>            Reporter: Alexei Gradinari
>            Severity: Minor
>
> As res_pjsip_nat rewrites contact's address, only the last Via header can contain the source address of registered endpoint.
> Also Call-Id header may contain the source address of registered endpoint.
> Added "via_addr", "via_port", "call_id" to contacts.
> Added new fields ViaAddress,CallID to AMI event ContactStatus.



--
This message was sent by Atlassian JIRA
(v6.2#6252)