I would like to discuss more about the removal of use_domain parameter. It is a common parameter name across many modules and it is a easy way to turn on/off multi-domain support (even done also for alias_db in default kamailio.cfg). I would like a coherent solution, like either removing this mod param everywhere or still keep it and also allow the per-function option to enable/disable. Moreover, changing the default in using the domain is going to break compatibility may result in unexpected behaviour without prior visible notice, as people can reuse an old config without use_domain parameter set, so no error is triggered, but behaviour is changed. So I think the default should be without domain (this is the default across kamailio modules) and a flag to enable domain based lookup. Keeping the use_domain won't change much the patch, the flags need to be initialized based on it: ``` if(use_domain) { flags |= **domain_lookup_flag** } ``` Travis reports that the inter-module API was broken -- app_lua doesn't compile as the function prototype for alias db lookup was changed. This is an easy fix, I can take care of. -- 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/pull/823#issuecomment-253757171

@khoegh pushed 2 commits. 456941e alias_db: Re-add module parameter use_domain c1e36d0 alias_db: Add flag to use domain to alias_db_lookup/alias_db_find -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/kamailio/kamailio/pull/823/files/bfa72df5327891dc5b8b296…

Closed #823. -- 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/pull/823#event-825624409

I did some testing and it seems to work 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/pull/823#issuecomment-254218447