Hello,
I was trying to write a small module using the pua api, unfortunatly I have found that there is no callback on publish reply (struct publ_info : publrpl_cb_t* cbrpl;) unlike what is described in the doc. (http://kamailio.org/docs/modules/devel/modules_k/pua.html#id2556188).
Maybe I have missed an old thread ex plaining that issue ? I have found that I can do something similar with register_puacb (modules_k/pua/send_publish.c:341) but in the doc it's said that the functio||n is for subscribe reply (it's why I was trying to add the callback in struct publ_info).
If it's a bug, I have already a patch to add the publrpl_cb_t in publ_info and I guess it's working but I have tested it only with my module. Please, tell me if you want that I open a new ticket in the bug tracker about it.
Thanks,
Hello,
It looks like it's missing from the code. You should report this in the tracker or send the patch to the list.
Cheers, Marius ________________________________________ From: sr-dev-bounces@lists.sip-router.org [sr-dev-bounces@lists.sip-router.org] On Behalf Of nikita [nikita@mbdsys.com] Sent: Tuesday, August 03, 2010 3:42 PM To: sr-dev@lists.sip-router.org Subject: [sr-dev] missing callback in struct publ_info (module pua)
Hello,
I was trying to write a small module using the pua api, unfortunatly I have found that there is no callback on publish reply (struct publ_info : publrpl_cb_t* cbrpl;) unlike what is described in the doc. (http://kamailio.org/docs/modules/devel/modules_k/pua.html#id2556188).
Maybe I have missed an old thread ex plaining that issue ? I have found that I can do something similar with register_puacb (modules_k/pua/send_publish.c:341) but in the doc it's said that the functio||n is for subscribe reply (it's why I was trying to add the callback in struct publ_info).
If it's a bug, I have already a patch to add the publrpl_cb_t in publ_info and I guess it's working but I have tested it only with my module. Please, tell me if you want that I open a new ticket in the bug tracker about it.
Thanks,
-- nikita
_______________________________________________ sr-dev mailing list sr-dev@lists.sip-router.org http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
On Thursday 05 August 2010, Marius Ovidiu Bucur wrote:
It looks like it's missing from the code. You should report this in the tracker or send the patch to the list.
Hi Nikita,
we just found out that somehow it was removed by purpose some years ago:
r2596, 2007-08-13
- restructured the reply callback system for Publish to make it simetric with the one for Subscribe - updated the pua_xmpp and pua_mi to the changes - enabled calling registered callback functions for FAKED_REPLY also - updated the presence_mwi module to a previous change made in presence
Still investigating..
Cheers,
Henning
-
Henning Westerholt -
Marius Ovidiu Bucur -
nikita