@fgaisnon: release process for v5.7.3 started earlier today and I didn't merge
before because @henningw's comment is rather confusing for me, requesting also
more time for others to review the PR.
As I understood from the description of the PR, this should be a fix for a
regression introduced by commit 091dc9a76bcec5c8a4bc73e863ed10b1b9d76c92, which
was done by @henningw. So handling early dialog BYEs was fine before that
commit, but his comment is about validity of sending BYE instead of CANCEL in
an early dialog.
BYE during early dialog has a different purpose than CANCEL, but that is not
the real matter here, handling early dialog BYEs was (supposed to be) supported
for long time by topos. Considering that I thought that @henningw's comment is
unrelated to the commit of the PR and since it was opened yesterday I haven't
had the time to review again the previous related commit and what this PR
brings to it, combined with the fact that I had no easy option to test early
dialog BYEs, I skipped it for 5.7.3.
Probably it will be merged soon if all ok with it and then backported,
afterwards you can use nightly builds of 5.7 series if you rely on Debian
packages, till 5.7.4 is out.
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3644#issuecomment-1816311617
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3644/c1816311...@github.com>
_______________________________________________
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org
