----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3207/#review10849 -----------------------------------------------------------
I can't really speak for the HEP parts nor all of this new asterisk 12 stuff, but the code looks nice and easy to read. /branches/12/CHANGES <https://reviewboard.asterisk.org/r/3207/#comment20444> An example hep.conf would be nice. /branches/12/res/res_hep.c <https://reviewboard.asterisk.org/r/3207/#comment20442> Blob. /branches/12/res/res_hep.c <https://reviewboard.asterisk.org/r/3207/#comment20443> Can this be false? /branches/12/res/res_hep.c <https://reviewboard.asterisk.org/r/3207/#comment20441> asterisk-12.x$ wgrep . aco_option_register | grep '"[a-z_]*_[a-z_]*"' | wc -l 41 asterisk-12.x$ wgrep . aco_option_register | grep '"[a-z-]*-[a-z-]*"' | wc -l 3 Underscores please? - wdoekes On Feb. 11, 2014, 4:16 a.m., Matt Jordan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3207/ > ----------------------------------------------------------- > > (Updated Feb. 11, 2014, 4:16 a.m.) > > > Review request for Asterisk Developers, Joshua Colp and Olle E Johansson. > > > Repository: Asterisk > > > Description > ------- > > This patch adds the following: > (1) A new module, res_hep, which implements a generic packet capture agent > for the Homer Encapsulation Protocol (HEP) version 3. Note that this code is > heavily based on a patch provided by Alexandr Dubovikov; I basically just > wrapped it up, added configuration via the configuration framework, and threw > in a taskprocessor. > (2) A new module, res_hep_pjsip, which performs packet capturing for the > PJSIP SIP stack. This is one of those modules that I think really showcases > how nice the new stack is - we're able to add a new module that inserts > itself into the stack and forwards the message traffic off to the res_hep > module without modifying the core parts of the stack itself. This means a > system administrator could load this at will on certain Asterisk systems and > - if the capturing isn't needed - unload it and keep the stack 'slim'. > > A few notes: > > * This code exists in the following branch: > http://svn.asterisk.org/svn/asterisk/team/mjordan/12-hep > * The code in the branch also contains a module for RTCP. While that actually > *does* send RTCP information over HEP, it does so as a JSON blob, which is > not super useful. It's an open question as to what the formatting should be, > i.e., a SNOM-esque encoding, RFC 6035, etc. I'm open to suggestions on this, > which is why I deferred that functionality for a later review. > * Much thanks to Alexandr for his Asterisk patch for this code and for a > *lot* of patience waiting for me to port it to 12/trunk. Due to some > dithering on my part, this has taken the better part of a year to port > forward (I still blame CDRs for the delay). > > > Diffs > ----- > > /branches/12/res/res_hep_pjsip.c PRE-CREATION > /branches/12/res/res_hep.exports.in PRE-CREATION > /branches/12/res/res_hep.c PRE-CREATION > /branches/12/include/asterisk/res_hep.h PRE-CREATION > /branches/12/CHANGES 407945 > > Diff: https://reviewboard.asterisk.org/r/3207/diff/ > > > Testing > ------- > > An automated test that emulates a SIP capture server was written and is up > for review here: https://reviewboard.asterisk.org/r/3206 > > This admittedly needs some *real* testing, as I have yet to stand up Kamailio > with HEP. I think the code is far enough along to get some eyes on it however. > > > Thanks, > > Matt Jordan > >
-- _____________________________________________________________________ -- 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