From notifications@github.com Thu Jun 18 15:03:43 2020 From: Victor Seva To: sr-dev@lists.kamailio.org Subject: [sr-dev] [kamailio/kamailio] topos: missing Record-Route on 200 OK leaving the server (downstream) (#2364) Date: Thu, 18 Jun 2020 06:03:31 -0700 Message-ID: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0143536380==" --===============0143536380== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable ### Description Seems that topos is not adding Record-Route headers in this scenario. ### Troubleshooting #### Reproduction ![topos](https://user-images.githubusercontent.com/63080/85021978-ea173900-b1= 72-11ea-95dc-2b3be27378e3.png) ![no_topos](https://user-images.githubusercontent.com/63080/85021983-eb486600= -b172-11ea-8383-816799e6251b.png) #### Log Messages I've added some more debug to topos: https://paste.debian.net/1152674/ Full [kamailio-lb.log](https://github.com/kamailio/kamailio/files/4798522/kam= ailio-lb.log) This is the part when processing of the 200 OK coming from PSTN(fake) ``` Jun 18 13:48:35 sp1 lb[14702]: DEBUG: topos [tps_msg.c:868]: tps_response_rec= eived(): handling incoming response Jun 18 13:48:35 sp1 lb[14702]: DEBUG: topos [tps_msg.c:376]: tps_pack_message= (): compacted headers - x_via1: [SIP/2.0/UDP 127.0.0.1:5064;branch=3Dz9hG4bKc= f3d.49d855e2b5f83b9dff62c8a6b0cf23f8.0](80) - x_via2: [](0) - x_vbranch1: [z9= hG4bKcf3d.49d855e2b5f83b9dff62c8a6b0cf23f8.0](46) Jun 18 13:48:35 sp1 lb[14702]: DEBUG: topos [tps_msg.c:485]: tps_pack_message= (): compacted headers - a_rr: [](0) - b_rr: [](0) - s_rr: [](0) Jun 18 13:48:35 sp1 lb[14702]: DEBUG: topos [tps_msg.c:490]: tps_pack_message= (): compacted headers - as_contact: [](0) - bs_contact: [](0) Jun 18 13:48:35 sp1 lb[14702]: DEBUG: topos [tps_storage.c:126]: tps_storage_= lock_get(): tps lock get: 220 Jun 18 13:48:35 sp1 lb[14702]: DEBUG: topos [tps_msg.c:889]: tps_response_rec= eived(): loaded dialog a_uuid [atpsh-5eeb547f-3970-3] Jun 18 13:48:35 sp1 lb[14702]: DEBUG: topos [tps_storage.c:391]: tps_storage_= link_msg(): downstream: yes - acontact: [] - bcontact: [sip:127.0.2.1:50603;t= ransport=3DUDP] Jun 18 13:48:35 sp1 lb[14702]: DEBUG: topos [tps_storage.c:138]: tps_storage_= lock_release(): tps lock release: 220 Jun 18 13:48:35 sp1 lb[14702]: DEBUG: topos [tps_msg.c:182]: tps_add_headers(= ): adding to headers(0) - [Via: SIP/2.0/UDP 127.0.0.1:5080;received=3D127.0.0= .1;branch=3Dz9hG4bKh2oO8aY.;rport=3D5080#015#012] Jun 18 13:48:35 sp1 lb[14702]: DEBUG: topos [tps_msg.c:910]: tps_response_rec= eived(): x_rr[] y_rr[] s_rr[,] Jun 18 13:48:35 sp1 lb[14702]: DEBUG: topos [tps_msg.c:182]: tps_add_headers(= ): adding to headers(0) - [Record-Route: ,#015#012] Jun 18 13:48:35 sp1 lb[14702]: DEBUG: topos [tps_msg.c:182]: tps_add_headers(= ): adding to headers(0) - [P-SR-XBranch: z9hG4bKcf3d.49d855e2b5f83b9dff62c8a6= b0cf23f8.0#015#012] Jun 18 13:48:35 sp1 lb[14702]: NOTICE: REPLY_FROM_OUTBOUND