[sr-dev] Re: [kamailio/kamailio] topos: add compatiblity for early in-dialog NOTIFY (PR #3627)

@mauritsderuiter95 commented on this pull request. 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/1710253923(a)github.com>