@mauritsderuiter95 commented on this pull request.
> @@ -1146,7 +1146,8 @@ int tps_db_load_branch(
if((get_cseq(msg)->method_id == METHOD_SUBSCRIBE)
|| ((get_cseq(msg)->method_id == METHOD_NOTIFY)
- && (msg->event && msg->event->len >
0))) {
+ && (msg->event && msg->event->len > 0
+ &&
strncmp(msg->event->body.s, "talk", 4) != 0))) {
Yes, the previous version accepted any NOTIFY with an event, and therefore set
`bInviteDlg = 0`. This doesn't work with the `Event: talk`, since in that
specific case topos needs to look for an INVITE dialog instead of a SUBSCRIBE
dialog. So it doesn't only accept event "talk", but the opposite: it accepts
any event except "talk".
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3627#discussion_r1380111256
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3627/review/1710253...@github.com>_______________________________________________
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org
