there is define in uac_reg.c (define reg_db_set_attr(attr, pos)) which may generate logs like this:
empty value not allowed for column[2]='l_domain' - ignoring record
really, record was added, so it should be converted to WARNING or it is neсessary add any more restrict logic
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/643
I guess you used it to refresh one record, because the process of loading the records at startup should ignore such records. reg_db_set_attr() triggers 'continue' in such case and it is used in a for-loop.
I am going to push a patch that will ignore also for the refresh operation. In that case reg_db_set_attr() was used alone, not within a loop.
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/643#issuecomment-222099080
it is startup error also
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/643#issuecomment-222100946
It is error indeed, but the record should not be added in that case, right?
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/643#issuecomment-222102302
record is added kamcmd> uac.reg_dump { l_uuid: 000112838 l_username: 000112838 l_domain: r_username: 000112838 ....... flags: 20 ...........
}
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/643#issuecomment-222103013
Obviously -- reg_db_set_attr() is a while loop itself so the continue applied there.
Pushed another patch -- can you test it? If all ok, I will backport.
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/643#issuecomment-222104525
it works as expected, thanks!
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/643#issuecomment-222106171
Closed #643.
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/643#event-674255882
Thanks for reporting and testing, I will backport.
While it could have been ok without domain, the other attributes are important and same check was done for all.
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/643#issuecomment-222106799
-
Daniel-Constantin Mierla -
Dmitri Savolainen