[asterisk-dev] [Code Review] 2811: New pjsip.conf.sample with 100% more pjsip

rnewton reviewboard at asterisk.org
Fri Aug 30 15:17:05 CDT 2013


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

(Updated Aug. 30, 2013, 8:17 p.m.)


Review request for Asterisk Developers.


Changes
-------

Attempted to remove all trailing whitespace, let's see.


Bugs: ASTERISK-22145
    https://issues.asterisk.org/jira/browse/ASTERISK-22145


Repository: Asterisk


Description
-------

pjsip.conf!

The file consists of two main bodies of text.
1. Manually written examples - fulfilling a variety of basic configuration scenarios. A few of which are detailed on the ASTERISK-22145 issue.
2. A full config option list - Output from a python script I wrote. It takes an xml config dump from Asterisk and parses the pjsip.conf config options out into the format you see in the file. (I may throw that script up here later after I improve it)

Before the examples there is a blurb talking about where the official documentation is and a brief security notice.

Notes on the thought process behind what we have so far:

* We want this to be easy to maintain

In writing this I found it's easy to turn it into exhaustive documentation and make it very redundant with our other documentation sources. For maintenance sake I trimmed it all back to be about as simple as possible and tried to aim it towards a "quick reference sheet", rather than "demonstrate every config option in action document".

The full option list is easy to update thanks to it being pulled from a script and not hand edited. 

* The sample files should start moving towards "sample" files and not be exhaustive documentation.
* The sample file is not a tutorial.

The examples provided in the file are geared towards someone who has already read the basics on the wiki[1] or elsewhere. The examples also helpful to serve as context for when you are looking at specific options through the cli config help or referencing the full option list in the bottom of the file. 

* The examples in the sample file should be simple for a user to understand and modify

The examples don't have every option listed under them because it makes it a bit confusing to look at. I've tried to arrange them realistically for basic configurations so that it would be easy for you to copy paste options from the big list at the bottom to the examples at the top.

* Every example is not explained in detail

I tried to pick out key points or options used in the examples, rather than just explaining them line by line (since that would lead to a lot of redundancy)

*** Don't provide feedback here for all the individual option names, synopsis, descriptions etc. That is all pulled from the XML config documentation. If you see errors in those areas please file a JIRA issue ***

Please provide feedback on the example configurations and text that is not in the big list of options at the bottom of the file. That is any obviously wrong configuration (option that couldn't be used in a section), horrible spelling, grammar or logic and flow problems. Suggestions for additional configuration examples that are crucial or essential would be very helpful, especially if you can write the example. Feedback is also very welcome on the formatting of the options list at the bottom. :)

Moving around all this text, I'm sure I missed quite a few things! Thanks in advance for the help!

[1]: https://wiki.asterisk.org/wiki/display/AST/Asterisk+12+Installation+and+Configuration


Diffs (updated)
-----

  /trunk/configs/pjsip.conf.sample 398037 

Diff: https://reviewboard.asterisk.org/r/2811/diff/


Testing
-------

I tested pretty much all the configurations in this file or pulled them from other peoples working configurations, there are probably a few option combinations and configurations that I missed, but added in.


Thanks,

rnewton

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130830/f4e46bef/attachment.htm>


More information about the asterisk-dev mailing list