New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SDP codec order #149

Closed
lschweizer opened this Issue May 11, 2016 · 1 comment

Comments

Projects
None yet
3 participants
@lschweizer
Contributor

lschweizer commented May 11, 2016

Hello,

I see that when the media server create a SDP answer the codec order returned is not the same that in the request, he use is internal codec order.

With some device, when they don’t have the same codec order that the media server it’s create an confusion and an no voice issue.

I have attached a simple patch to solve this issue.
codec_order_diff.txt

Without the patch:

U 95.128.80.40:2727 -> 95.128.80.40:2427
CRCX 404261276 mobicents/ivr/$@95.128.80.40:2427 MGCP 1.0
C: 1
M: sendonly

o=- 227700400 2 IN IP4 62.2.46.12
s=-
c=IN IP4 62.2.46.12
t=0 0
m=audio 29880 RTP/AVP 0 8 100
a=fmtp:100 0-15
a=rtpmap:100 telephone-event/8000
a=silenceSupp:off - - - -
a=ptime:20


U 95.128.80.40:2427 -> 95.128.80.40:2727
200 404261276 Success
I:1
Z:mobicents/ivr/1@95.128.80.40:2427

v=0
o=- 1462907743868 1 IN IP4 95.128.80.40
s=Mobicents Media Server
c=IN IP4 95.128.80.40
t=0 0
m=audio 65534 RTP/AVP 8 0
c=IN IP4 95.128.80.40
a=sendrecv
a=rtcp:65535 IN IP4 95.128.80.40
a=ptime:20
a=rtpmap:0 pcmu/8000
a=rtpmap:8 pcma/8000
a=ssrc:1489405285 cname:lPBSariY7gH07fDK

with the patch

U 95.128.80.40:2727 -> 95.128.80.40:2427
CRCX 453018054 mobicents/ivr/$@95.128.80.40:2427 MGCP 1.0
C: 1
M: sendonly

o=- 227700400 2 IN IP4 62.2.46.12
s=-
c=IN IP4 62.2.46.12
t=0 0
m=audio 29880 RTP/AVP 0 8 100
a=fmtp:100 0-15
a=rtpmap:100 telephone-event/8000
a=silenceSupp:off - - - -
a=ptime:20


U 95.128.80.40:2427 -> 95.128.80.40:2727
200 453018054 Success
I:1
Z:mobicents/ivr/1@95.128.80.40:2427

v=0
o=- 1462962783310 1 IN IP4 95.128.80.40
s=Mobicents Media Server
c=IN IP4 95.128.80.40
t=0 0
m=audio 65534 RTP/AVP 0 8
c=IN IP4 95.128.80.40
a=sendrecv
a=rtcp:65535 IN IP4 95.128.80.40
a=ptime:20
a=rtpmap:0 pcmu/8000
a=rtpmap:8 pcma/8000
a=ssrc:1871274706 cname:sX6dNGGuH8WetjGw
@deruelle

This comment has been minimized.

Member

deruelle commented May 11, 2016

@lschweizer Thanks for the contribution. However, could you do a formal pull request as described in our Open Source Playbook Dev Process Section as well as sign the Contributor Licensor Agreement ?

@deruelle deruelle added the bug label May 11, 2016

@deruelle deruelle added this to the 4.2.0 milestone May 11, 2016

@hrosa hrosa closed this in b1da9cd May 12, 2016

@hrosa hrosa removed the Peer Review label May 12, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment