henningw left a comment (kamailio/kamailio#4415)

> Hello, had a quick look why it works with SQL storage => because it just sets 
> empty "smethod" for all NOTIFYs, not just for "talk" event: [1]
> 
> Any reason why the fix should not look the same for htable storage?
> 
> Thank you, Stefan
> 
> [1]
> 
> [kamailio/src/modules/topos/tps_storage.c](https://github.com/kamailio/kamailio/blob/5d73826c9b3b7ececf01358a7396e5e395f15872/src/modules/topos/tps_storage.c#L1251)
> 
> Line 1251 in 
> [5d73826](/kamailio/kamailio/commit/5d73826c9b3b7ececf01358a7396e5e395f15872)
>  } else if(get_cseq(msg)->method_id == METHOD_NOTIFY) {

It would be good to have the same behaviour for all topos backends regarding 
the NOTIFY handling. The problem was fixed in #3627 for topos with several 
commits. We probably should port this fixed to topos_htable, and also 
topos_redis if necessary. A PR would be of course great, as mentioned.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/4415#issuecomment-3371273892
You are receiving this because you are subscribed to this thread.

Message ID: <kamailio/kamailio/issues/4415/[email protected]>
_______________________________________________
Kamailio - Development Mailing List -- [email protected]
To unsubscribe send an email to [email protected]
Important: keep the mailing list in the recipients, do not reply only to the 
sender!

Reply via email to