[OpenSIPS-Devel] Proposal for change in add_diversion function

Bogdan-Andrei Iancu bogdan at opensips.org
Wed May 30 17:56:28 CEST 2012


Hi Saúl,

Logically speaking, what you say makes sense - better use the current 
RURI and not the original one.

As per tracker, please proceed and apply the patch .

Thanks and regards,

Bogdan-Andrei Iancu
OpenSIPS Founder and Developer
http://www.opensips-solutions.com


On 05/29/2012 03:18 PM, Saúl Ibarra Corretgé wrote:
> Hi all,
>
> I'd like to make a change to the add_diversion function of the diversions module and I'd like some feedback, in case my proposed change could cause trouble to anyone.
>
> Right now the aforementioned function inserts the original request URI of the message as the indication of who made the diversion. It does so as follows:
>
> uri =&msg->first_line.u.request.uri;
>
> This means that if the request URI was modified since the message was received, the new URI won't be set in the diversion header, the old one will. In cases where ENUM is involved, for example, one could want to see the real URI after the ENUM resolution.
>
> Would anyone oppose to setting the *current* RURI instead? That is, take the URI from msg->parsed_uri.
>
> As an alternative, we could keep the current behavior and add a module parameter (diverter_avp) where one could set the URI he wanted to see in the diversion header, and use the current approach if the AVP is not set.
>
> Thoughts?
>
>
> Thanks and regards,
>
> --
> Saúl Ibarra Corretgé
> AG Projects
>
>
>
>
> _______________________________________________
> Devel mailing list
> Devel at lists.opensips.org
> http://lists.opensips.org/cgi-bin/mailman/listinfo/devel
>



More information about the Devel mailing list