[asterisk-bugs] [JIRA] (ASTERISK-27448) [patch] Add ability to send progress inband by setting a channel variable instead of fixed config per endpoint

[nappsoft (JIRA)]

    [ https://issues.asterisk.org/jira/browse/ASTERISK-27448?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=241342#comment-241342 ] 

nappsoft commented on ASTERISK-27448:
-------------------------------------

Hey Joshua. I did some tests on a Asterisk11/chan_sip based system with progressinband not set. chan_sip sends the ringing notification automatically inband after audio has been sent as early media (for example with Progress() followed by Plaback with the noanswer flag set).

So this seems to be a behavior change when migrating from chan_sip to chan_pjsip which I am able to work around with my patch. (Of course adding the chan_sip behavior to chan_pjsip would be a cleaner solution)

> [patch] Add ability to send progress inband by setting a channel variable instead of fixed config per endpoint
> --------------------------------------------------------------------------------------------------------------
>
>                 Key: ASTERISK-27448
>                 URL: https://issues.asterisk.org/jira/browse/ASTERISK-27448
>             Project: Asterisk
>          Issue Type: Improvement
>      Security Level: None
>          Components: Channels/chan_pjsip
>    Affects Versions: 13.18.2
>            Reporter: nappsoft
>            Assignee: nappsoft
>            Severity: Minor
>              Labels: patch, pjsip
>         Attachments: progressinband3.diff
>
>
> I've faced a problem with several voip switches when using 180 Ringing messages after announcements sent over 183 Session Progress with SDP (no audible progress indication after Progress followed by Playback). The solution would be to use inband_progress=yes but this way progress will be sent as rtp for each call, so as well in cases in which it is not necessary.
> So my idea was to set a channel variable PROGRESSINBAND to 1 in situations in which we use Progress() followed by Playback().
> I've added some code to chan_pjsip.c which seems to work. I've opened a ticket here for two reasons: one: the "hack" could be usefull for someone else and second: I wanted to ask whether it is safe to do it the way I solved this or whether I'd need to lock channel->session->channel or something else first?



--
This message was sent by Atlassian JIRA
(v6.2#6252)