< Previous by Date Date Index Next by Date >
  Thread Index  

[reSIProcate] Fwd: Commit 3871


---------- Forwarded message ----------
From: Scott Godin <slgodin@xxxxxxxxxxxx>
Date: Wed, 16 Feb 2005 15:34:49 -0500
Subject: RE: Commit 3871
To: Scott Godin <slgodin@xxxxxxxxxxxx>, Kenneth Ho <kenho@xxxxxxxxxxxxxx>
Cc: derek <derek@xxxxxxxx>, Jason Fischl <jason@xxxxxxxxxxxxxx>


OK - I commited a fix - so that the CSeq method is changed to INVITE before
trying to match using the MergeRequestKey.

It seems to work now.

Derek/Jason - check out the change I made.  Let me know if you think there
is a more elegant way to handle this.  I was considering a CancelKey map -
used only for Cancel -> dialogset matching and not using the
MergedRequestKey - but this solution seems to work too.


-----Original Message-----
From: Scott Godin
Sent: Wednesday, February 16, 2005 12:59 PM
To: 'Kenneth Ho'; Scott Godin
Cc: 'derek'; 'Jason Fischl'
Subject: RE: Commit 3871

Oops - sorry.

You are right I changed Cancel to Dialogset mapping.

I just reviewed the code - The problem is in the matching of CSEQ header
from MergeRequestKey.  I thought I tested it.... I apologize.

I'll fix this ASAP and commit.

-----Original Message-----
From: Kenneth Ho [mailto:kenho@xxxxxxxxxxxxxx]
Sent: Wednesday, February 16, 2005 12:37 PM
To: Scott Godin
Subject: Commit 3871

Hi Scott,

Commit 3871 broke our UAS,  I am not sure what the problem is (CANCEL
<-> DialogSet matching?).
Though it might be broken by something else, I am investigating into it
as well.
Please help and take a look.
Thanks~

Ken