Summary:	ASTERISK-11344: [patch] realtime Queue members seem not to work using res_config_curl
Reporter:	jmls (jmls)	Labels:
Date Opened:	2008-01-31 08:55:48.000-0600	Date Closed:	2008-02-01 16:33:28.000-0600
Priority:	Minor	Regression?	No
Status:	Closed/Complete	Components:	Resources/res_config_curl
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:	( 0) res_config_curl.patch
Description:	Perhaps I am missing something really stupid here, but I can't seem to get realtime queuemembers to work. I added some diagnostics to res_config_curl to report the data, you can see it below. The member always has an invalid interface I am using the 1.4 version of res_config_curl. I can't test use 1.6, as it doesn't even return the queue, using "queue show ROCK" ****** ADDITIONAL INFORMATION ****** data from bash: root@victor asterisk# /usr/bin/curl http://192.168.6.6/myapp/queuemembers-multi?queue_name=ROCK uniqueid=22&queue_name=rock&interface=local%2F5707@agentq%2Fn&penalty=0&membername=jmls33&paused=no data from CLI: show queue ROCK ROCK         has 0 calls (max unlimited) in 'rrmemory' strategy (0s holdtime), W:0, C:0, A:0, SL:0.0% within 60s   Members:      jmls33 (realtime) (Invalid) has taken no calls yet   No Callers victor*CLI> adding ast_verbose(VERBOSE_PREFIX_1 "res_config_curl: Name: %s (%s)\n",key,pair); gives the following: res_config_curl: Name: uniqueid (22) res_config_curl: Name: queue_name (rock) res_config_curl: Name: interface (local/5707@agentq/n) res_config_curl: Name: penalty (0) res_config_curl: Name: membername (jmls33) res_config_curl: Name: paused (no)
Comments:	By: jmls (jmls) 2008-01-31 13:33:59.000-0600 ok, found out why 1.6 is not working for me: Change ast_str_set(&query, 0, "${CURL(%s,", url); to ast_str_set(&query, 0, "${CURL(%s/single,", url); (this makes 1.6 consistent with the 1.4 backport, and in line with what we discussed during development and testing) By: jmls (jmls) 2008-01-31 13:37:11.000-0600 there are some further problems with 1.6 vs 1.4: I use exactly the same config with 1.4 and 1.6 (I make install from the two different source directories, removing the modules as appropriate) 1.4: victor*CLI> queue show ROCK ROCK         has 0 calls (max unlimited) in 'rrmemory' strategy (0s holdtime), W:0, C:0, A:0, SL:0.0% within 60s   Members:      jmls44 (realtime) (Invalid) has taken no calls yet   No Callers victor*CLI> 1.6: victor*CLI> queue show ROCK ROCK         has 0 calls (max unlimited) in 'ringall' strategy (0s holdtime), W:0, C:0, A:0, SL:0.0% within 60s   Members:      jmls44 (realtime) (Invalid) has taken no calls yet   No Callers [Jan 31 19:39:33] WARNING[32253]: app_queue.c:844 create_queue_member: No location at interface '' By: jmls (jmls) 2008-01-31 14:05:58.000-0600 found the problem with 1.4: the returned string must contain the word "interface%20LIKE=". So for documentation purposes, the data in the notes should be uniqueid=22&queue_name=rock&&interface%20LIKE=local%2F5707@agentq%2Fn&penalty=0&membername=jmls33&paused=no By: jmls (jmls) 2008-01-31 14:43:07.000-0600 1.6 also works with setting "interface like" By: jmls (jmls) 2008-02-01 04:10:30.000-0600 added patch to make 1.6 work the same way as the 1.4 version By: Digium Subversion (svnbot) 2008-02-01 12:42:43.000-0600 Repository: asterisk Revision: 101873 U   trunk/res/res_config_curl.c ------------------------------------------------------------------------ r101873 | tilghman | 2008-02-01 12:42:37 -0600 (Fri, 01 Feb 2008) | 7 lines Fix multi, when using the LIKE query. (closes issue ASTERISK-11344) Reported by: jmls Patches:       res_config_curl.patch uploaded by jmls (license 141) Tested by: jmls ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=101873 By: Digium Subversion (svnbot) 2008-02-01 16:33:28.000-0600 Repository: asterisk Revision: 101988 _U  team/tilghman/res_odbc_astobj2/ U   team/tilghman/res_odbc_astobj2/apps/app_authenticate.c U   team/tilghman/res_odbc_astobj2/apps/app_voicemail.c U   team/tilghman/res_odbc_astobj2/configs/res_odbc.conf.sample U   team/tilghman/res_odbc_astobj2/configure U   team/tilghman/res_odbc_astobj2/configure.ac U   team/tilghman/res_odbc_astobj2/include/asterisk/autoconfig.h.in U   team/tilghman/res_odbc_astobj2/res/res_config_curl.c U   team/tilghman/res_odbc_astobj2/res/res_odbc.c ------------------------------------------------------------------------ r101988 | tilghman | 2008-02-01 16:33:28 -0600 (Fri, 01 Feb 2008) | 88 lines Merged revisions 101819,101821,101823-101824,101869,101873,101895,101943 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r101819 | russell | 2008-02-01 11:26:31 -0600 (Fri, 01 Feb 2008) | 12 lines Merged revisions 101818 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r101818 | russell | 2008-02-01 11:23:47 -0600 (Fri, 01 Feb 2008) | 4 lines Don't overwrite the last character of a line if it's not a newline.  This would happen if the last line in the file doesn't have a newline. (pointed out by Qwell) ........ ................ r101821 | russell | 2008-02-01 11:28:06 -0600 (Fri, 01 Feb 2008) | 8 lines Blocked revisions 101820 via svnmerge ........ r101820 | russell | 2008-02-01 11:27:02 -0600 (Fri, 01 Feb 2008) | 1 line off by one error ........ ................ r101823 | qwell | 2008-02-01 11:44:32 -0600 (Fri, 01 Feb 2008) | 3 lines Move an feof() call to before the fgets(). This would have exited the loop early if you had an authentication file with no newline at the end. ................ r101824 | tilghman | 2008-02-01 12:08:44 -0600 (Fri, 01 Feb 2008) | 2 lines Clarify the pooling functionality by changing the config file keyword ................ r101869 | qwell | 2008-02-01 12:24:52 -0600 (Fri, 01 Feb 2008) | 1 line Comparison, not set :)  Thanks mvanbaak. ................ r101873 | tilghman | 2008-02-01 12:45:31 -0600 (Fri, 01 Feb 2008) | 7 lines Fix multi, when using the LIKE query. (closes issue ASTERISK-11344) Reported by: jmls Patches:       res_config_curl.patch uploaded by jmls (license 141) Tested by: jmls ................ r101895 | tilghman | 2008-02-01 13:44:39 -0600 (Fri, 01 Feb 2008) | 10 lines Merged revisions 101894 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r101894 | tilghman | 2008-02-01 13:36:12 -0600 (Fri, 01 Feb 2008) | 2 lines Change detection of getifaddrs to use AST_C_COMPILE_CHECK, backported from trunk (as suggested by kpfleming) ........ ................ r101943 | tilghman | 2008-02-01 16:12:55 -0600 (Fri, 01 Feb 2008) | 16 lines Merged revisions 101942 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r101942 | tilghman | 2008-02-01 15:54:28 -0600 (Fri, 01 Feb 2008) | 8 lines Fix the VM_DUR variable for forwarded voicemail, and fixed several other bugs while I'm in the area. (closes issue ASTERISK-11095) Reported by: jamessan Patches:       20071226__bug11615__2.diff.txt uploaded by Corydon76 (license 14) Tested by: Corydon76, jamessan ........ ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=101988