Ok my latest makes me think this is an issue in db_cassandra.

when I add the following debug line it displays as expected.

496 if(_c) {

497 for(int i=0; i< _nc; i++) {

498 sp.column_names.push_back(_c[i]->s);

499 LM_DBG("Query col: %s\n", _c[i]->s);

500 LM_DBG("JAY Query col: %.*s\n", _c[i]->len, _c[i]->s);

501 }

502 LM_DBG("get %d columns\n", _nc);

503 sp.__isset.column_names = true; // set

so _c[i]->s is just going to the end of the string, instead of respecting _c[i]->len

While I COULD work out a fix for this using sprintf or similar, it would probably not be in keeping with kamaiio convention.
( since im not experienced with kamailio code )
Also, im not sure what other places I would miss.

can someone more experienced with kamailio take a look and propose a patch for me to test.

Thanks

Jay

On 9 December 2013 21:34, jay binks <jaybinks@gmail.com> wrote:

kamailio -v
version: kamailio 4.2.0

im still digging, but my current hunch is that my prior diagnosis may be wrong, and this might be in db_cassandra ...
still digging, but I will update the list with my findings if they are of interest.
On 9 December 2013 21:26, Daniel-Constantin Mierla <miconda@gmail.com> wrote:
What version are you using?

Daniel

On 09/12/13 12:17, jay binks wrote:
So when Im trying to use is_subscriber & load_credentials together, It seems I have either hit a bug or just a bad config ( but I dont think its the latter )

My symptom is that only the field on the far right of "load_credentials" ever returns the correct value, the other fields are always null.

My config has this setting :

modparam("auth_db", "load_credentials", "field1;field2")

and I can see the following debug when running kamailio -ddddd

0(14174) DEBUG: auth_db [authdb_mod.c:291]: w_is_subscriber(): uri [sip: 1112@192.168.173.5] table [subscriber] flags [1]

0(14174) DEBUG: db_cassandra [dbcassa_base.cpp:807]: db_cassa_query(): query table=subscriber

0(14174) DEBUG: db_cassandra [dbcassa_table.c:461]: dbcassa_db_get_table(): found dbname=kamailio, table=subscriber

0(14174) DEBUG: db_cassandra [dbcassa_base.cpp:351]: cassa_constr_key(): query col = username

0(14174) DEBUG: db_cassandra [dbcassa_base.cpp:351]: cassa_constr_key(): query col = domain

0(14174) DEBUG: db_cassandra [dbcassa_base.cpp:376]: cassa_constr_key(): key = 1112 192.168.173.5

0(14174) DEBUG: db_cassandra [dbcassa_base.cpp:499]: cassa_translate_query(): Query col: field2

0(14174) DEBUG: db_cassandra [dbcassa_base.cpp:499]: cassa_translate_query(): Query col: field1; field2

0(14174) DEBUG: db_cassandra [dbcassa_base.cpp:501]: cassa_translate_query(): get 2 columns

0(14174) DEBUG: <core> [db_res.c:118]: db_new_result(): allocate 56 bytes for result set at 0x7fc6ce770e78

if I had to put money on where the bug is, I would look in parse_aaa_pvs in authdb_mod.c

however, an initial glance revealed nothing glaring, also this file was last changed in 2013-01-20 ( and that was only a move from modules_k )

SURELY I cant be the only person wanting to use this for more than one field ?

This code looks like its almost the same as when first committed in August 2011 ....

Id love someone to see if they can confirm this and point me in the right direction ( if this is my fault ) or lets get this fixed, if its a bug :)

--
Sincerely

Jay
_______________________________________________
SIP Express Router (SER) and Kamailio (OpenSER) - sr-users mailing list
sr-users@lists.sip-router.org
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-users
-- Daniel-Constantin Mierla - http://www.asipto.com http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda
_______________________________________________
SIP Express Router (SER) and Kamailio (OpenSER) - sr-users mailing list
sr-users@lists.sip-router.org
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-users
--
Sincerely

Jay

--
Sincerely

Jay