Re: [OpenSIPStack] 200 ok in ProxySession::OnSessionEvent

[<jo...@op...>]

Commited a patch for this.  Thanks for the bug report.

Gustavo Curetti wrote:
> Hi Joegen,
>  
> It fails in SBCRoutingHandler::RouteProxyRequest() because the condition
>  
> if( !request.IsCancel() && !request.IsAck() )
>  
> is TRUE for the 200 OK response and never get the condition
>  
> }else if( !request.IsRequest() )
> {
>      OnRecordRouteShift( request );
> }Regards
>  
> Gustavo
>
>
>
>   
>> Date: Thu, 29 May 2008 10:07:14 +0800> To: cur...@gm...; ope...@li...> From: joe...@gm...> Subject: Re: [OpenSIPStack] 200 ok in ProxySession::OnSessionEvent> > Hi Gustavo,> > Can you tell where it fails in SBCRoutingHandler::RouteProxyRequest() ?> > The only purpose of OnCheckRoutePolicy() for responses is to modify the > Record-Routes in cases where the UAS and UAC transaction is traversing > different NICs. OpenSBC must change the Record-Route before it sends > the response to the caller.> > Joegen> > > Gustavo Curetti wrote:> > Hi Joegen> > > > I don't understand why in ProxySession::OnSessionEvent() the method OnCheckRoutePolicy is called in case EventFinalResponse:> > > > case EventFinalResponse:> > OnCheckRoutePolicy( validatedRequest ); ???> > OnFinalResponse( validatedRequest );> > break;> > > > throwing this trace:> > > > *** PROXY *** Not Allowing relay to > > > > Thanks for your help Gustavo> > _________________________________________________________________> > Ingresá ya a MSN Deportes y enterate de las últimas novedades del mundo deportivo.> > http://msn.foxsports.com/fslasc/> > -------------------------------------------------------------------------> > This SF.net email is sponsored by: Microsoft> > Defy all challenges. Microsoft(R) Visual Studio 2008.> > http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/> > _______________________________________________> > opensipstack-devel mailing list> > ope...@li...> > https://lists.sourceforge.net/lists/listinfo/opensipstack-devel> >> >> > > > > > -------------------------------------------------------------------------> This SF.net email is sponsored by: Microsoft> Defy all challenges. Microsoft(R) Visual Studio 2008.> http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/> _______________________________________________> opensipstack-devel mailing list> ope...@li...> https://lists.sourceforge.net/lists/listinfo/opensipstack-devel
>>     
> _________________________________________________________________
> Descargá ya gratis y viví la experiencia Windows Live.
> http://www.descubrewindowslive.com/latam/index.html
> -------------------------------------------------------------------------
> This SF.net email is sponsored by: Microsoft
> Defy all challenges. Microsoft(R) Visual Studio 2008.
> http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
> _______________________________________________
> opensipstack-devel mailing list
> ope...@li...
> https://lists.sourceforge.net/lists/listinfo/opensipstack-devel
>
>
>
>