[asterisk-commits] pjsip/attribute passthrough/speex h263 h264: Disable direct ... (testsuite[master])

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Thu Oct 27 20:38:12 CDT 2016


Anonymous Coward #1000019 has submitted this change and it was merged. ( https://gerrit.asterisk.org/4213 )

Change subject: pjsip/attribute_passthrough/speex_h263_h264: Disable direct media.
......................................................................


pjsip/attribute_passthrough/speex_h263_h264: Disable direct media.

The fixing of ASTERISK-26423 has resulted in direct media being
attempted for cases where it would not have previously occurred.
This is because the RTP native bridge check was getting all
configured codecs (not negotiated). If video was configured then
no direct media would be attempted, even if video was not
negotiated. This is no longer the case so direct_media needs
to be explicitly disabled.

ASTERISK-26423

Change-Id: Iacdc75c43978cd1ffce5f01c792bb73805ed2fe3
---
M tests/channels/pjsip/sdp_offer_answer/attribute_passthrough/speex_h263_h264/configs/ast1/pjsip.conf
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Richard Mudgett: Looks good to me, but someone else must approve
  Anonymous Coward #1000019: Verified
  Joshua Colp: Looks good to me, approved



diff --git a/tests/channels/pjsip/sdp_offer_answer/attribute_passthrough/speex_h263_h264/configs/ast1/pjsip.conf b/tests/channels/pjsip/sdp_offer_answer/attribute_passthrough/speex_h263_h264/configs/ast1/pjsip.conf
index 8e9a0a9..b9a553e 100644
--- a/tests/channels/pjsip/sdp_offer_answer/attribute_passthrough/speex_h263_h264/configs/ast1/pjsip.conf
+++ b/tests/channels/pjsip/sdp_offer_answer/attribute_passthrough/speex_h263_h264/configs/ast1/pjsip.conf
@@ -7,6 +7,7 @@
 type=endpoint
 context=default
 media_address=127.0.0.1
+direct_media=no
 
 [phoneA](endpoint-template)
 disallow=all

-- 
To view, visit https://gerrit.asterisk.org/4213
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iacdc75c43978cd1ffce5f01c792bb73805ed2fe3
Gerrit-PatchSet: 1
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>



More information about the asterisk-commits mailing list