Re: [OpenSIPS-Devel] [opensips] RTPPROXY module: compatibility with topology hiding (#237)

2014-05-21 Thread Nick Altmann
Closed #237. --- Reply to this email directly or view it on GitHub: https://github.com/OpenSIPS/opensips/pull/237#event-123212752___ Devel mailing list Devel@lists.opensips.org http://lists.opensips.org/cgi-bin/mailman/listinfo/devel

[OpenSIPS-Devel] [opensips] RTPPROXY module: compatibility with topology hiding (#237)

2014-05-20 Thread Nick Altmann
RTPPROXY module: workaround to run rtpproxy when we use dialog topology hiding and thus we have not 2nd via in reply It fixes autobridging for rtpproxy when using topology hiding. The problem is reply doesn#39;t have second via. We used this patch on production system about 7 months without any

Re: [OpenSIPS-Devel] [opensips] RTPPROXY module: compatibility with topology hiding (#237)

2014-05-20 Thread Walter Doekes
} else { - LM_ERR(can't extract 2nd via found reply\n); + if (parse_headers(msg, HDR_VIA2_F, 0) != -1 You could make the `} else {` above an `} else if {`, that

Re: [OpenSIPS-Devel] [opensips] RTPPROXY module: compatibility with topology hiding (#237)

2014-05-20 Thread Walter Doekes
@@ -3229,6 +3229,8 @@ force_rtp_proxy(struct sip_msg* msg, char* str1, char* str2, char *setid, char * struct force_rtpp_args args; struct force_rtpp_args *ap; union sockaddr_union to; + struct socket_info *si; + struct dlg_cell * dlg; Please stay consistent with