[asterisk-dev] [Code Review] Remove need for registration strings in sip.conf

[Dmitry Andrianov]

Out of curiosity, how does this work with realtime peers?
Will there be some syntax telling asterisk "pre-load peers named AAA and BBB" to force their registration (given they have register=yes in options) ?

-----Original Message-----
From: asterisk-dev-bounces at lists.digium.com [mailto:asterisk-dev-bounces at lists.digium.com] On Behalf Of Nick Lewis
Sent: Thursday, June 17, 2010 7:02 PM
To: David Vossel
Cc: Asterisk Developers; Nick Lewis
Subject: Re: [asterisk-dev] [Code Review] Remove need for registration strings in sip.conf


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

(Updated 2010-06-17 10:01:52.894512)


Review request for Asterisk Developers and David Vossel.


Changes
-------

Addressed comments from dvossel


Summary
-------

This change adds the ability to specify the sip registration expiry and registrar host to the peer configuration items. This permits all registration details that are available to the registration string to be also available to the peer definitions. The users of sip.conf can enable registration by setting register=yes in the peer and do not need to concern themselves with the strange format of the registration strings

Suggest inclusion in 1.8


Diffs (updated)
-----

  trunk/channels/chan_sip.c 268968 
  trunk/channels/sip/config_parser.c 268968 
  trunk/channels/sip/include/sip.h 268968 
  trunk/configs/sip.conf.sample 268968 

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


Testing
-------

Compile, run and confirm in cli sip show registry


Thanks,

Nick


-- 
_____________________________________________________________________
-- Bandwidth and Colocation Provided by http://www.api-digital.com --

asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-dev