From: Nick French Date: Tue, 13 Oct 2020 17:15:28 +0000 (-0500) Subject: res_pjsip_session: Restore calls to ast_sip_message_apply_transport() X-Git-Tag: 17.9.0-rc1~9 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=f7835bcf13ebf9340dc41abd16ab36334f6534ac;p=thirdparty%2Fasterisk.git res_pjsip_session: Restore calls to ast_sip_message_apply_transport() Commit 44bb0858cb3ea6a8db8b8d1c7fedcfec341ddf66 ("debugging: Add enough to choke a mule") accidentally removed calls to ast_sip_message_apply_transport when it was attempting to just add debugging code. The kiss of death was saying that there were no functional changes in the commit comment. This makes outbound calls that use the 'flow' transport mechanism fail, since this call is used to relay headers into the outbound INVITE requests. ASTERISK-29124 #close Change-Id: I0f3e32c2e8ac415e30b1d29966d75a1546f0526a --- diff --git a/res/res_pjsip_session.c b/res/res_pjsip_session.c index 9178174dfb..a4236be32e 100644 --- a/res/res_pjsip_session.c +++ b/res/res_pjsip_session.c @@ -4340,6 +4340,8 @@ static void handle_outgoing_request(struct ast_sip_session *session, pjsip_tx_da SCOPE_ENTER(3, "%s: Method is %.*s\n", ast_sip_session_get_name(session), (int) pj_strlen(&req.method.name), pj_strbuf(&req.method.name)); + ast_sip_message_apply_transport(session->endpoint->transport, tdata); + AST_LIST_TRAVERSE(&session->supplements, supplement, next) { if (supplement->outgoing_request && does_method_match(&req.method.name, supplement->method)) { supplement->outgoing_request(session, tdata); @@ -4364,6 +4366,8 @@ static void handle_outgoing_response(struct ast_sip_session *session, pjsip_tx_d ast_sip_session_get_name(session)); } + ast_sip_message_apply_transport(session->endpoint->transport, tdata); + AST_LIST_TRAVERSE(&session->supplements, supplement, next) { if (supplement->outgoing_response && does_method_match(&cseq->method.name, supplement->method)) { supplement->outgoing_response(session, tdata);