[OpenSIPS-Devel] [OpenSIPS/opensips] 7c1e78: Fixed missing traced SIP messages.

Bogdan-Andrei Iancu bogdan at opensips.org
Thu Aug 3 07:48:19 EDT 2017


  Branch: refs/heads/2.3
  Home:   https://github.com/OpenSIPS/opensips
  Commit: 7c1e78e2ed2b83f35c82bc9152fe669d9331a75c
      https://github.com/OpenSIPS/opensips/commit/7c1e78e2ed2b83f35c82bc9152fe669d9331a75c
  Author: Bogdan-Andrei Iancu <bogdan at opensips.org>
  Date:   2017-08-03 (Thu, 03 Aug 2017)

  Changed paths:
    M modules/siptrace/siptrace.c
    M modules/tm/t_cancel.c
    M modules/tm/t_fwd.c
    M modules/tm/t_hooks.h
    M modules/tm/t_lookup.c
    M modules/tm/t_reply.c
    M modules/tm/timer.c

  Log Message:
  -----------
  Fixed missing traced SIP messages.

The following SIP messages were not traced by SIP trace:
* 100 trying generated by OpenSIPS
* incoming ACK for negative replies
* outgoing ACK to negative replies
* incoming replies to CANCEL requests
* incoming request retransmissions
* outgoing request retransmissions
* outgoing reply retransmissions

The fix introduced two new TM callback to get access to the lowest level in TM message handling:
	* matching a SIP message to the transaction
	* sending a SIP message related to the transaction

(cherry picked from commit 6eec7f95025f0bda0e1dfa764430934a73e90e87)


  Commit: 0594318dc73ccdc5ed2b1eab8032b2ebe28539fa
      https://github.com/OpenSIPS/opensips/commit/0594318dc73ccdc5ed2b1eab8032b2ebe28539fa
  Author: Bogdan-Andrei Iancu <bogdan at opensips.org>
  Date:   2017-08-03 (Thu, 03 Aug 2017)

  Changed paths:
    M modules/siptrace/siptrace.c
    M modules/siptrace/siptrace.h

  Log Message:
  -----------
  Propagate the tracing context via function param.

Inside SIPtrace module propagate the tracing context as function param instead of relying on the processing context - there are cases were we have no processing context (like retransmissions).

(cherry picked from commit e59a7399e982ef58a7a12ffb53d8aa8432f537ab)


Compare: https://github.com/OpenSIPS/opensips/compare/9667332c36bf...0594318dc73c


More information about the Devel mailing list