RestComm / Restcomm-Connect

The Open Source Cloud Communications Platform
http://www.restcomm.com/
GNU Affero General Public License v3.0
240 stars 214 forks source link

BS-282 imcomming call duration should not contain Ringing duration of… #2985

Closed xhoaluu closed 5 years ago

xhoaluu commented 6 years ago

… outbound call

What this PR does / why we need it: Incoming leg rings for 1m while waiting for the outbound leg to answer. We include that 1m on the incoming leg duration, but from Twilio example, it seems they don't do it. And the biggest problem happens when outgoing leg doesn’t answer, in this case, basically, we charged the incoming leg with a ringing duration for a call that didn’t happen.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged): Fixes # https://telestax.atlassian.net/browse/BS-282 Special notes for your reviewer:

gvagenas commented 5 years ago

Merged