[Home]

Summary:ASTERISK-04014: [patch] Forwarding Context
Reporter:dsandras (dsandras)Labels:
Date Opened:2005-04-29 05:56:02Date Closed:2005-05-01 14:34:43
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) chan_sip.c.patch
Description:Several companies have asked us to have a different call context for call forward. For example, some users could forward calls from their SIP Phone to their cellular, and some others could not, even if they are allowed to call cellulars directly.

Here is a patch that implements that.
Comments:By: Kevin P. Fleming (kpfleming) 2005-04-29 10:38:47

We don't add new features to the stable branch. Please test and regenerate your patch against CVS HEAD and we can consider it for inclusion.

Also, you have not noted whether you have a disclaimer on file; without one, we cannot accept your patch, even against CVS HEAD.

By: Olle Johansson (oej) 2005-05-01 12:13:01

Why are you implementing this in the user? We receive calls from the user, places calls to peers. It's more useful in a peer, since we never call the user and receives a call forward from their phone.

Can't you solve this in the dial plan?

By: dsandras (dsandras) 2005-05-01 14:07:18

You are right, it should be solved in the peers.

I don't see how to solve that in the dialplan with so many possibilities. With that patch you can precisely define the set of destinations to which calls can be forwarded, independantly from what the user can call. Perhaps it doesn't look interesting for you, in that case, just close the bug. But it has been requested several times.

I can not port the patch to CVS unfortunately. There is no disclaimer.

By: Kevin P. Fleming (kpfleming) 2005-05-01 14:34:17

Without a disclaimer, we cannot accept your code, for any branch. This bug will be closed, and the patch can be reposted in the future if you can provide a disclaimed version against the HEAD branch.