[reSIProcate] Transport logging patch

[Scott Godin]

Hi Palladin,

I have taken this patch, modified it and committed it.  Thanks for
contributing.  Here's a I highlight of what I changed:
-removed use of the term Decorator in the name - these callbacks are for
logging only, not message modification - renamed to SipMessageLoggingHandler
-renamed the virtual print methods to be more descriptive: outboundMessage
and inboundMessage
-added source and destination parameters to inbound messages to be
consistent
-made SipMessage arguments const - to indicate you cannot modify the
SipMessages - only examine / log them
-removed static use - each Transport now can carry it's own handler.  You
can set a single handle in SipStack::setTransportSipMessageLoggingHandler
  - this implementation is flexible and allows a single handler per stack
instance or handler per transport instance - it is not locked to one
handler for all stack instances
-added ability to turn on INFO level SipMessage logging to repro
configuration

Regards,
Scott


 -added ability to install a SipMessageLoggingHandler that will get
called for every inbound and outbound
  SIP Message on a particular transport - work started by Palladin
 -added SipStack::setTransportSipMessageLoggingHandler to set logger
on all added transports
 -added new Repro setting EnableSipMessageLogging to enable INFO level
SipMessage logging
 -ReproRunner: enasure we pass PrivateKeyPassPhrase when custom
private key file is provided
 -ReproRunner: if transport fails to create - don't bother processing it further





Modified:

   - main/repro/ReproRunner.cxx<http://svn.resiprocate.org/viewsvn/resiprocate/main/repro/ReproRunner.cxx?r1=11150&r2=11151&diff_format=l>
   - main/repro/repro.config<http://svn.resiprocate.org/viewsvn/resiprocate/main/repro/repro.config?r1=11150&r2=11151&diff_format=l>
   - main/resip/stack/SipStack.cxx<http://svn.resiprocate.org/viewsvn/resiprocate/main/resip/stack/SipStack.cxx?r1=11150&r2=11151&diff_format=l>
   - main/resip/stack/SipStack.hxx<http://svn.resiprocate.org/viewsvn/resiprocate/main/resip/stack/SipStack.hxx?r1=11150&r2=11151&diff_format=l>
   - main/resip/stack/Transport.cxx<http://svn.resiprocate.org/viewsvn/resiprocate/main/resip/stack/Transport.cxx?r1=11150&r2=11151&diff_format=l>
   - main/resip/stack/Transport.hxx<http://svn.resiprocate.org/viewsvn/resiprocate/main/resip/stack/Transport.hxx?r1=11150&r2=11151&diff_format=l>
   - main/resip/stack/TransportSelector.cxx<http://svn.resiprocate.org/viewsvn/resiprocate/main/resip/stack/TransportSelector.cxx?r1=11150&r2=11151&diff_format=l>
   - main/resip/stack/UdpTransport.cxx<http://svn.resiprocate.org/viewsvn/resiprocate/main/resip/stack/UdpTransport.cxx?r1=11150&r2=11151&diff_format=l>
   - main/resip/stack/ssl/DtlsTransport.cxx<http://svn.resiprocate.org/viewsvn/resiprocate/main/resip/stack/ssl/DtlsTransport.cxx?r1=11150&r2=11151&diff_format=l>



On Tue, Apr 8, 2014 at 11:04 AM, palladin <p-aladin at yandex.ru> wrote:

> Hi,
> please ignore previous patch, it has small mistake.
> Here is a new one.
>
> Sincerely, tez.
>
> _______________________________________________
> resiprocate-devel mailing list
> resiprocate-devel at resiprocate.org
> https://list.resiprocate.org/mailman/listinfo/resiprocate-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://list.resiprocate.org/pipermail/resiprocate-devel/attachments/20140422/17a87282/attachment.htm>