Re: [SR-Users] permissions module address_reload not working

2010-07-02 Thread Andrei Pelinescu-Onciul
On Jul 01, 2010 at 13:38, Juha Heinanen wrote: > Daniel-Constantin Mierla writes: > > > Juha, I just committed on master and tested over MI where works fine. > > Can you do some tests over rpc? If all ok then can be backported. > > it would be nice if i could, but after trying to cherry-pick my

Re: [SR-Users] permissions module address_reload not working

2010-07-01 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > No time to do real tests, if you can run some and report back, it would > be great. Once some testing is done, can be ported to 3.0. daniel, i tested address_reload, trusted_reload, and sht_reload mi commands over xmlrpc and they all worked fine. also 'ctl mi

Re: [SR-Users] permissions module address_reload not working

2010-07-01 Thread Daniel-Constantin Mierla
On 7/1/10 5:54 PM, Daniel-Constantin Mierla wrote: On 7/1/10 4:39 PM, Juha Heinanen wrote: Daniel-Constantin Mierla writes: this is another issue ... the mi commands were designed to be run from special processes, created by mi transport modules (mi_fifo, mi_datagram, etc). There are speci

Re: [SR-Users] permissions module address_reload not working

2010-07-01 Thread Daniel-Constantin Mierla
On 7/1/10 4:39 PM, Juha Heinanen wrote: Daniel-Constantin Mierla writes: this is another issue ... the mi commands were designed to be run from special processes, created by mi transport modules (mi_fifo, mi_datagram, etc). There are special init functions called for these processes. Howe

Re: [SR-Users] permissions module address_reload not working

2010-07-01 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > this is another issue ... the mi commands were designed to be run from > special processes, created by mi transport modules (mi_fifo, > mi_datagram, etc). There are special init functions called for these > processes. However, with 3.0 xmlrpc module can call t

Re: [SR-Users] permissions module address_reload not working

2010-07-01 Thread JR Richardson
> -Original Message- > From: Juha Heinanen [mailto:j...@tutpro.com] > Sent: Thursday, July 01, 2010 1:43 AM > To: JR Richardson; SR-Users > Subject: Re: [SR-Users] permissions module address_reload not working > > Juha Heinanen writes: > > > revert

Re: [SR-Users] permissions module address_reload not working

2010-07-01 Thread Daniel-Constantin Mierla
On 7/1/10 1:18 PM, Juha Heinanen wrote: Daniel-Constantin Mierla writes: Juha, I just committed on master and tested over MI where works fine. Can you do some tests over rpc? If all ok then can be backported. daniel, i deleted whole sr git repo and downloaded a new one. after tha

Re: [SR-Users] permissions module address_reload not working

2010-07-01 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > Juha, I just committed on master and tested over MI where works fine. > Can you do some tests over rpc? If all ok then can be backported. daniel, i deleted whole sr git repo and downloaded a new one. after that i did address_reload tests. it works from comma

Re: [SR-Users] permissions module address_reload not working

2010-07-01 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > Juha, I just committed on master and tested over MI where works fine. > Can you do some tests over rpc? If all ok then can be backported. it would be nice if i could, but after trying to cherry-pick my msilo commit to sr_3.0, i got errors about debian-lenny fil

Re: [SR-Users] permissions module address_reload not working

2010-07-01 Thread Daniel-Constantin Mierla
On 7/1/10 10:44 AM, Juha Heinanen wrote: Daniel-Constantin Mierla writes: I guess you use this command via rpc interface, not via mi. yes, i use 'ctl mi address_reload', noticed that it didn't work and then went and fixed it. Looking at the code, the mi init child is not done

Re: [SR-Users] permissions module address_reload not working

2010-07-01 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > I guess you use this command via rpc interface, not via mi. yes, i use 'ctl mi address_reload', noticed that it didn't work and then went and fixed it. > Looking at > the code, the mi init child is not done for rpc processes, resulting in > some mi commands n

Re: [SR-Users] permissions module address_reload not working

2010-07-01 Thread Daniel-Constantin Mierla
Hello Juha, On 7/1/10 8:28 AM, Juha Heinanen wrote: JR Richardson writes: Well, you sure are a smart fella. I reverted the patch and reloading the address table works as expected now. For clarification I was testing with kamailio-3.0.2_src_2010-06-30_a6141a.tar.gz snapshot. Let me know w

Re: [SR-Users] permissions module address_reload not working

2010-06-30 Thread Juha Heinanen
Juha Heinanen writes: > reverting the patch thus most likely reintroduced this other bug. > perhaps the bug that you saw has something to do with the last sentence > of the commit. jr, did you try with db_mode=1? it works ok here. -- juha ___ SIP Ex

Re: [SR-Users] permissions module address_reload not working

2010-06-30 Thread Juha Heinanen
JR Richardson writes: > Well, you sure are a smart fella. I reverted the patch and reloading > the address table works as expected now. For clarification I was > testing with kamailio-3.0.2_src_2010-06-30_a6141a.tar.gz snapshot. > Let me know when you have the updated fix and I'll do sime testin

Re: [SR-Users] permissions module address_reload not working

2010-06-30 Thread JR Richardson
On Wed, Jun 30, 2010 at 4:51 PM, Daniel-Constantin Mierla wrote: > Hello, > > On 6/30/10 10:17 PM, JR Richardson wrote: >> >> Hi All, >> >> I'm running kamailio 3.0.2. >> > > hmm, checking the logs I see changes done lately that introduced some bugs: > http://git.sip-router.org/cgi-bin/gitweb.cgi?

Re: [SR-Users] permissions module address_reload not working

2010-06-30 Thread Daniel-Constantin Mierla
Hello, On 6/30/10 10:17 PM, JR Richardson wrote: Hi All, I'm running kamailio 3.0.2. hmm, checking the logs I see changes done lately that introduced some bugs: http://git.sip-router.org/cgi-bin/gitweb.cgi?p=sip-router;a=commitdiff;h=b0091da84ee5bbbc18cb799eefc497be707ab0f9 I could spot

[SR-Users] permissions module address_reload not working

2010-06-30 Thread JR Richardson
Hi All, I'm running kamailio 3.0.2. # - permissions params - modparam("permissions", "db_url", "mysql://openserro:opense...@localhost/openser") #modparam("permissions", "db_mode", 1) snip in route block: if (!allow_source_address_group()) { sl_send_reply("403", "