[Freeswitch-dev] uuid in mod_spandsp_fax.c

[Steve Underwood]

I can add an additional user controlled opaque pointer to each instance 
of spandsp's logging, and extend the callbacks to pass that pointer to 
the application. Most other things in spandsp pass opaque pointers in 
callbacks, so it would kind of complete the set.

Steve


On 03/28/2012 01:18 AM, Anthony Minessale wrote:
> I think you would need to change spandsp itself to allow a private
> pointer to pass to the logging callback or you're out of luck.
>
>
>
> On Tue, Mar 27, 2012 at 10:56 AM, Michael Collins<msc at freeswitch.org>  wrote:
>> *SIGH* evidently there's a magic keystroke that means "send" in GMail and I
>> accidentally pressed it.
>>
>> As I was saying, it looks like this function is only called 6 times:
>>
>> <139>:grep -n spanfax_log_message mod_spandsp_fax.c
>> 262:void spanfax_log_message(int level, const char *msg)
>> 716:        span_log_set_message_handler(&fax->logging,
>> spanfax_log_message);
>> 717:        span_log_set_message_handler(&t30->logging,
>> spanfax_log_message);
>> 772:            span_log_set_message_handler(&t38->logging,
>> spanfax_log_message);
>> 773:            span_log_set_message_handler(&t30->logging,
>> spanfax_log_message);
>> 827:     span_log_set_message_handler(&pvt->t38_gateway_state->logging,
>> spanfax_log_message);
>> 828:     span_log_set_message_handler(&pvt->t38_core->logging,
>> spanfax_log_message);
>>
>> I'd wager that each of those calls to the logging function would have
>> knowledge of the sesssion. I'd recommend adding a 3rd argument to the
>> logging function. In this case, I'd just add a const char *uuid and then
>> change each call to that function to supply the UUID as the 3rd argument. If
>> there is no knowledge of the session or uuid then just put an empty string
>> as the argument.
>>
>> In any case, happy coding! :)
>>
>> -MC
>>
>>
>> On Tue, Mar 27, 2012 at 1:15 AM, Anita Hall<anita.hall at simmortel.com>
>> wrote:
>>> Yes Mitch, I did that but the Fax internal messages are printed by this
>>> line which has no knowledge of the session.
>>>    switch_log_printf(SWITCH_CHANNEL_LOG, fs_log_level, "%s", msg);
>>>
>>> I do not think, it can print the uuid without knowing the session ?
>>>
>>> Here is a log snippet to show uuid prefixed at other places except line
>>> 286 of mod_spandsp_fax.c shown above
>>>
>>> 2012-03-27 13:40:38.604877 [DEBUG] mod_spandsp_fax.c:286 FLOW T.30   1...
>>> ....= Extension indicator: Set
>>> 2012-03-27 13:40:38.604877 [DEBUG] mod_spandsp_fax.c:286 FLOW T.30   ....
>>> ..0.= Compressed/uncompressed mode: Compressed
>>> 2012-03-27 13:40:38.604877 [DEBUG] mod_spandsp_fax.c:286 FLOW T.30   ....
>>> .1..= Error correction mode (ECM): ECM
>>> 2012-03-27 13:40:38.604877 [DEBUG] mod_spandsp_fax.c:286 FLOW T.30   ....
>>> 0...= Frame size: 256 octets
>>> 2012-03-27 13:40:38.604877 [DEBUG] mod_spandsp_fax.c:286 FLOW T.30   .1..
>>> ....= T.6 coding: Set
>>> 2012-03-27 13:40:38.604877 [DEBUG] mod_spandsp_fax.c:286 FLOW T.30   0...
>>> ....= Extension indicator: Not set
>>> 2012-03-27 13:40:38.604877 [DEBUG] mod_spandsp_fax.c:286 FLOW T.30
>>> Selected compression T.6 (3)
>>> 46736448-77e4-11e1-855c-b3286880c45b 2012-03-27 13:40:38.604877 [DEBUG]
>>> mod_spandsp_fax.c:327 === Negotiation Result
>>> =======================================================
>>> 46736448-77e4-11e1-855c-b3286880c45b 2012-03-27 13:40:38.604877 [DEBUG]
>>> mod_spandsp_fax.c:328 Remote station id:
>>> 46736448-77e4-11e1-855c-b3286880c45b 2012-03-27 13:40:38.604877 [DEBUG]
>>> mod_spandsp_fax.c:329 Local station id:  Sangoma Fax Ident
>>> 46736448-77e4-11e1-855c-b3286880c45b 2012-03-27 13:40:38.604877 [DEBUG]
>>> mod_spandsp_fax.c:330 Transfer Rate:     4800
>>> 46736448-77e4-11e1-855c-b3286880c45b 2012-03-27 13:40:38.604877 [DEBUG]
>>> mod_spandsp_fax.c:332 ECM status         on
>>> 46736448-77e4-11e1-855c-b3286880c45b 2012-03-27 13:40:38.604877 [DEBUG]
>>> mod_spandsp_fax.c:333 remote country:
>>> 46736448-77e4-11e1-855c-b3286880c45b 2012-03-27 13:40:38.604877 [DEBUG]
>>> mod_spandsp_fax.c:334 remote vendor:
>>> 46736448-77e4-11e1-855c-b3286880c45b 2012-03-27 13:40:38.604877 [DEBUG]
>>> mod_spandsp_fax.c:335 remote model:
>>> 46736448-77e4-11e1-855c-b3286880c45b 2012-03-27 13:40:38.604877 [DEBUG]
>>> mod_spandsp_fax.c:337
>>> =========================================================================
>>>
>>>
>>> regards,
>>> Anita
>>>
>>> _________________________________________________________________________
>>> Professional FreeSWITCH Consulting Services:
>>> consulting at freeswitch.org
>>> http://www.freeswitchsolutions.com
>>>
>>> 
>>> 
>>>
>>> Official FreeSWITCH Sites
>>> http://www.freeswitch.org
>>> http://wiki.freeswitch.org
>>> http://www.cluecon.com
>>>
>>> FreeSWITCH-dev mailing list
>>> FreeSWITCH-dev at lists.freeswitch.org
>>> http://lists.freeswitch.org/mailman/listinfo/freeswitch-dev
>>> UNSUBSCRIBE:http://lists.freeswitch.org/mailman/options/freeswitch-dev
>>> http://www.freeswitch.org
>>>
>>
>> _________________________________________________________________________
>> Professional FreeSWITCH Consulting Services:
>> consulting at freeswitch.org
>> http://www.freeswitchsolutions.com
>>
>> 
>> 
>>
>> Official FreeSWITCH Sites
>> http://www.freeswitch.org
>> http://wiki.freeswitch.org
>> http://www.cluecon.com
>>
>> FreeSWITCH-dev mailing list
>> FreeSWITCH-dev at lists.freeswitch.org
>> http://lists.freeswitch.org/mailman/listinfo/freeswitch-dev
>> UNSUBSCRIBE:http://lists.freeswitch.org/mailman/options/freeswitch-dev
>> http://www.freeswitch.org
>>
>
>