@toharish commented on this pull request.


In src/modules/topos/topos_mod.c:

> @@ -490,10 +503,9 @@ int tps_msg_received(sr_event_param_t *evp)
 			}
 		}
 		dialog = (get_to(&msg)->tag_value.len>0)?1:0;
-		if(dialog) {
-			/* dialog request */
-			tps_request_received(&msg, dialog);
-		}
+		
+		tps_request_received(&msg, dialog);
+		

yes, it's required to mask the Call-ID for the inial Request.
Please refer to the changes in tps_msg.c function tps_request_received where it is masking the Call-ID and then returning back when dialog=0

if(dialog==0) {
		tps_mask_callid(msg);
		/* nothing to do for initial request other than Call-ID mask */
		return 0;
	}

This could have been done by checking (dialog==0) and calling tps_mask_callid(msg) directly in topos_mod.c insted of calling tps_request_received(&msg, dialog).

Any Request which is entered from the Upstream Call-ID mask is done
Any Request which is sent to Upstream Call-ID Un-mask is done

Any response which is entered from the Upstream Call-ID mask is done
Any response which is sent to Upstream Call-ID Un-mask is done


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <kamailio/kamailio/pull/3334/review/1266135078@github.com>