[asterisk-dev] [Code Review] 3207: HEP: Add a Homer Encapsulation Protocol (HEP) v3 capture agent module and a packet logger for PJSIP

Matt Jordan reviewboard at asterisk.org
Mon Feb 10 21:58:31 CST 2014

This is an automatically generated e-mail. To reply, visit:

Review request for Asterisk Developers, Joshua Colp, Lorenzo Miniero, and Olle E Johansson.

Repository: Asterisk


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:
* 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).


  /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/


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.


Matt Jordan

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140211/8a60b618/attachment-0001.html>

More information about the asterisk-dev mailing list