opensipstack-atlsipdevel

[ATLSIP] Logging

[OSS F. A. <ope...@op...>]

A previous message in the OpenSipStack forum discussed re-enabling logging for ATLSIP:
  < http://www.opensourcesip.org:8080/clearspacex/message/4130#4130 >

The proposed solution, as I read it, improves the log appearance by adding CRLF after ">>>" and "<<<" lines.  It doesn't actually add any information to the log.

To add CRLF and the incoming packet to the log, add
	traceStream << "\r\n";
	traceStream << msg->AsString();
after the existing 'traceStream << "<<< "'  statement in SIPTransportManager::OnProcessInbound in SIPTransportManager.cxx.  The new code was added at line 1134 in my copy of the file from 2008-06-13.

(Of course, you could always gang the new output onto the existing traceStream statement.)

Similarly, you can add
	traceStream << "\r\n";
	traceStream << packet->AsString();
after the existing 'traceStream << ">>> "' statement in SIPTransportManager::OnProcessOutbound.  The new code was added at line 1233 in my copy.

(The referenced file is part of the OpenSipStack project.  You, will of course, need to be able to build OpenSipStack and ATLSIP.)

The change might make the log too verbose once a connection is made.  Since I haven't been able to get a connection working yet, I need the additional information to track down the problem.

Re: [ATLSIP] Logging

[OSS F. A. <ope...@op...>]

I have patched CVS.  Thanks for reporting it.

Joegen


> {quote:title=optotronic wrote:}{quote}
> A previous message in the OpenSipStack forum discussed re-enabling logging for ATLSIP:
>   < http://www.opensourcesip.org:8080/clearspacex/message/4130#4130 >
> 
> The proposed solution, as I read it, improves the log appearance by adding CRLF after ">>>" and "<<<" lines.  It doesn't actually add any information to the log.
> 
> To add CRLF and the incoming packet to the log, add
> 	traceStream << "\r\n";
> 	traceStream << msg->AsString();
> after the existing 'traceStream << "<<< "'  statement in SIPTransportManager::OnProcessInbound in SIPTransportManager.cxx.  The new code was added at line 1134 in my copy of the file from 2008-06-13.
> 
> (Of course, you could always gang the new output onto the existing traceStream statement.)
> 
> Similarly, you can add
> 	traceStream << "\r\n";
> 	traceStream << packet->AsString();
> after the existing 'traceStream << ">>> "' statement in SIPTransportManager::OnProcessOutbound.  The new code was added at line 1233 in my copy.
> 
> (The referenced file is part of the OpenSipStack project.  You, will of course, need to be able to build OpenSipStack and ATLSIP.)
> 
> The change might make the log too verbose once a connection is made.  Since I haven't been able to get a connection working yet, I need the additional information to track down the problem.