[reSIProcate] Question about ClientSubscription and onRequestRetry

[Jeremy Geras]

We're in the middle of re-basing from the latest release, and found that this change by Francis prevents onNewSubscription from getting called at all in the normal case (UAC sends the SUBSCRIBE, UAS responds as normal).



Are we missing something?



Thanks,

Jeremy

________________________________
From: resiprocate-devel-bounces at resiprocate.org [resiprocate-devel-bounces at resiprocate.org] on behalf of Scott Godin [sgodin at sipspectrum.com]
Sent: Wednesday, April 17, 2013 2:19 PM
To: Francis Joanis
Cc: resiprocate-devel
Subject: Re: [reSIProcate] Question about ClientSubscription and onRequestRetry

Sounds right to me - the mOnNewSubscriptionCalled check will ensure we don't call onNewSubscription twice.

Scott

On Wed, Apr 17, 2013 at 4:03 PM, Francis Joanis <francis.joanis at gmail.com<mailto:francis.joanis at gmail.com>> wrote:
Hi guys,

For the following scenario:

- DUM UAC sends SUBSCRIBE to UAS
- UAS isn't responding and UAC times out
- On the UAC, onRequestRetry gets called and returns 0 (to retry ASAP)
- DUM retries
- UAS replies as usual (normal SIP flow)

In that case, we would expect onNewSubscription(client) to be called when the NOTIFY comes in once the 2nd SUBSCRIBE works but it doesn't...

It looks like the following line (91) in ClientSubscription.cxx:

if(!mOnNewSubscriptionCalled && !getAppDialogSet()->isReUsed())

should actually read

if(!mOnNewSubscriptionCalled && getAppDialogSet()->isReUsed())

without the ! in front of the second check.

I made a quick test and it looks like it resolves the issue, but I wanted to run it by you guys before submitting a fix. What do you think?

Thanks,
Francis

_______________________________________________
resiprocate-devel mailing list
resiprocate-devel at resiprocate.org<mailto:resiprocate-devel at resiprocate.org>
https://list.resiprocate.org/mailman/listinfo/resiprocate-devel

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://list.resiprocate.org/pipermail/resiprocate-devel/attachments/20130801/a02295a4/attachment.htm>