[asterisk-dev] [Code Review] 4328: res_pjsip: Document transport selection process

[Joshua Colp]

> On Jan. 13, 2015, 3:48 p.m., jbigelow wrote:
> > * Under 'URI Parsing' it states: "For the purposes of transport selection the transport parameter is examined." I suggest clarifing that the transport parameter being spoke of is the 'transport' parameter for endpoint types in pjsip.conf.
> > 
> > * Under 'DNS SRV Resolution (If host portion is not an IP address)' it states: "This means that the original URI *must* include..." I believe 'must' should be bold without the surrounding '*'.
> > 
> > * Under 'Best Configuration Strategies':
> >     * 'IPv4 Only (Single Interface)' it states "Configure a wildcard transport." Does this mean not specifying the 'transport' parameter on endpoints? I suggest to clarify this and to include an endpoint in the configuration showing that the 'transport' parameter is not specified (assuming thats what this example is about).
> >     * 'IPv4 Only (Multiple Interfaces)': it states "This allows each transport to be specified on endpoints...". Some clarification on whether this means specifying a single transport using the 'transport' parameter or specifying multiple transports using multiple 'transport' parameters for a single endpoint? I also suggest to include configuration of endpoints showing the 'transport' parameter being used for the example.
> >     * I suggest to either add additional examples with TLS transports. One showing the recommended way to configure endpoints where some use UDP and others use TLS.
> 
> Joshua Colp wrote:
>     That's not the transport being spoken of. It literally means a SIP URI such as sip:1000 at 127.0.0.1;transport=tcp
> 
> Joshua Colp wrote:
>     I've tweaked things some, added some endpoints, and referenced the applicable section that will be used depending on their configuration.
> 
> jbigelow wrote:
>     * Under 'Best Configuration Strategies':
>         * 'IPv4 Only (Single Interface)' it states "Since there is only one transport configured this will always be used. For requests to this endpoint the logic in section 3a will be used." which needs updated due to the last couple changes.

Tweaked to be more specific.


- Joshua


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4328/#review14168
-----------------------------------------------------------


On Jan. 12, 2015, 1:33 p.m., Joshua Colp wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/4328/
> -----------------------------------------------------------
> 
> (Updated Jan. 12, 2015, 1:33 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> The transport selection process of PJSIP (and by extension some of our own logic) can be a dark dark thing. To help illuminate what happens I have created a wiki page[1] which goes through (based on the message type) the process by which a transport is chosen and how it can potentially change.
> 
> Stuff to look at:
> 1. Is this detailed enough?
> 2. Can you follow it? If not, how could it be made clearer?
> 3. Are there additional common issues that should be covered?
> 
> [1] https://wiki.asterisk.org/wiki/display/~jcolp/Transport+Selection
> 
> 
> Diffs
> -----
> 
> 
> Diff: https://reviewboard.asterisk.org/r/4328/diff/
> 
> 
> Testing
> -------
> 
> I opened the wiki page. It opened.
> 
> 
> Thanks,
> 
> Joshua Colp
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20150114/66619346/attachment.html>