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

DTMF issue with dynamic payload id different that 101 #300

Closed
lschweizer opened this Issue Nov 16, 2016 · 2 comments

Comments

Projects
3 participants
@lschweizer
Contributor

lschweizer commented Nov 16, 2016

Hello,

if we send to the media server a SDP that contains an other payload id that the 101 for the DTMF , the media server is not processing them correctly and send back a SDP without any DTMF .

in the example bellow, we send DTMF event with the payload id 96

U 95.128.80.40:2727 -> 95.128.80.40:2427
CRCX 705691991 mobicents/ivr/$@95.128.80.40:2427 MGCP 1.0
C: 2
M: sendrecv

v=0.
o=hiQ9200 134920161010173114 390856728 IN IP4 62.180.246.11.
s=Phone Call via hiQ9200 SIPCA.
c=IN IP4 62.180.246.11.
t=0 0.
m=audio 14250 RTP/AVP 8 0 18 96.
a=rtpmap:0 PCMU/8000.
a=fmtp:0 vad=no.
a=rtpmap:8 PCMA/8000.
a=fmtp:8 vad=no.
a=rtpmap:18 G729/8000.
a=fmtp:18 annexb=yes.
a=rtpmap:96 telephone-event/8000.
a=fmtp:96 0-15.
a=sqn: 0.
a=cdsc: 1 image udptl t38.
a=sendrecv.

U 95.128.80.40:2427 -> 95.128.80.40:2727
200 705691991 Success
I:2fe
Z:mobicents/ivr/2@95.128.80.40:2427

v=0
o=- 1479248759269 1 IN IP4 95.128.80.40
s=Mobicents Media Server
c=IN IP4 95.128.80.40
t=0 0
m=audio 65532 RTP/AVP 8 0 18
c=IN IP4 95.128.80.40
a=sendrecv
a=rtcp:65533 IN IP4 95.128.80.40
a=ptime:20
a=rtpmap:0 pcmu/8000
a=rtpmap:18 g729/8000
a=rtpmap:8 pcma/8000
a=ssrc:3406629907 cname:iSi78BpwemUu9Mze

with debug info,

23:31:53,449 INFO [RestCommMediaServer] Global hearbeat is still alive
23:32:30,205 DEBUG [MgcpProvider] Parsing message: CRCX 705691992 mobicents/ivr/$@95.128.80.40:2427 MGCP 1.0
C: 3
M: sendrecv

v=0
o=hiQ9200 134920161010173114 390856728 IN IP4 62.180.246.11
s=Phone Call via hiQ9200 SIPCA
c=IN IP4 62.180.246.11
t=0 0
m=audio 14250 RTP/AVP 8 0 18 96
a=rtpmap:0 PCMU/8000
a=fmtp:0 vad=no
a=rtpmap:8 PCMA/8000
a=fmtp:8 vad=no
a=rtpmap:18 G729/8000
a=fmtp:18 annexb=yes
a=rtpmap:96 telephone-event/8000
a=fmtp:96 0-15
a=sqn: 0
a=cdsc: 1 image udptl t38
a=sendrecv

23:32:30,234 INFO [Transaction] tx=705691992 Started, message= CRCX mobicents/ivr/$@95.128.80.40:2427, call agent = /95.128.80.40:2727
23:32:30,235 DEBUG [EndpointQueue] Endpoint mobicents/ivr/1 taken (free=49)
23:32:30,238 DEBUG [ResourcesPool] Allocated remote connection [pool size:50, free:49]
23:32:30,239 DEBUG [ResourcesPool] Allocated DTMF Detector [pool size:50, free:49]
23:32:30,268 DEBUG [RtpConnectionImpl] Audio formats: RTPFormats{8 AudioFormat[pcma,8000,8,mono],0 AudioFormat[pcmu,8000,8,mono],18 AudioFormat[g729,8000,mono]}
23:32:30,270 INFO [Transaction] tx=705691992 was executed normaly
23:32:53,449 INFO [RestCommMediaServer] Global hearbeat is still alive

BR

Laurent

@hrosa

This comment has been minimized.

Collaborator

hrosa commented Nov 21, 2016

@lschweizer thank you for reporting this issue.

Seems we need to add better support for dynamic payload negotiation. Are you interested in contributing to this one?

@lschweizer

This comment has been minimized.

Contributor

lschweizer commented Nov 22, 2016

Hello,

yes, I will try.

Laurent

@lschweizer lschweizer referenced this issue Jan 19, 2017

Closed

Issue300 #347

@hrosa hrosa added this to the 5.2.0 milestone Jan 24, 2017

@hrosa hrosa modified the milestones: 5.2.0, 6.0.0 Feb 14, 2017

@hrosa hrosa modified the milestones: 6.1.0, 6.0.0 Feb 24, 2017

@hrosa hrosa modified the milestones: 6.1.0, 7.0.0, 7.0, 8.0 Mar 6, 2017

@hrosa hrosa modified the milestones: 8.0, 7.x-sprint10 Dec 13, 2017

hrosa added a commit that referenced this issue Dec 13, 2017

#300 Added support for dynamic payload types as referred by RTP Audio…
…-Video profile. Takes into account clock rates declared in SDP offer.

@hrosa hrosa closed this in 4c70783 Dec 13, 2017

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